Front page | perl.perl5.porters |
Postings from July 2016
From: Dave Mitchell
July 7, 2016 08:46
Message ID: 20160707084644.GM17054@iabyn.com
On Sun, Jul 03, 2016 at 11:19:59PM +0200, demerphq wrote:
> On 30 Jun 2016 23:01, "Karl Williamson" <firstname.lastname@example.org> wrote:
> > Commit 8bf4c4010cc474d4000c2a8c78f6890fa5f1e577 added this function, but
> marked it as deprecated. I suspect this is wrong, and it is not supposed
> to be deprecated.
> Depends on your definition of wrong..
> It was deliberate but perhaps misguided...
> I wanted to express that the function was available but with no future
> commitments at all, including removal.
> Feel free to change it to something more appropriate.
I've just pushed this:
Author: David Mitchell <email@example.com>
AuthorDate: Thu Jul 7 09:40:22 2016 +0100
This function was recently added, but marked as deprecated in embed.fnc.
Based on this thread:
It looks like the 'M' (may change) flag is sufficient to capture what was
intended. Removing the 'D' avoids every usage in core emitting a warning
to STDERR during build, while it still adds a caution to the entry in
NOTE: this function is experimental and may change or be
removed without notice.
Affected files ...
diff --git a/embed.fnc b/embed.fnc
index fe5eb68..88a70f9 100644
@@ -2774,7 +2774,7 @@ Apod |void |hv_assert |NN HV *hv
ApdR |SV* |hv_scalar |NN HV *hv
-ApdRMD |SV* |hv_bucket_ratio|NN HV *hv
+ApdRM |SV* |hv_bucket_ratio|NN HV *hv
ApoR |I32* |hv_riter_p |NN HV *hv
ApoR |HE** |hv_eiter_p |NN HV *hv
Apo |void |hv_riter_set |NN HV *hv|I32 riter
"I do not resent criticism, even when, for the sake of emphasis,
it parts for the time with reality".
-- Winston Churchill, House of Commons, 22nd Jan 1941.