Marcus Holland-Moritz wrote:

On 2005-06-23, at 14:42:21 +0100, Nicholas Clark wrote:

On Fri, Jun 17, 2005 at 09:55:37PM -0600, Jim Cromie wrote:

I left the api-pod as is, since folks shouldnt be encouraged to choose 5.9 over compatibility.
But isn't that what Devel::PPPort is for? Providing edible cake.

Yes. :-)

Marcus

heres that patch again, in 2 pieces.

1.  bzipped changes New -> Newx, New? -> Newx?
also includes macros for old x-less New, so that XS code continues to work wo src changes.
Theres XS out there that doesnt package ppport.h

2. patch to API pod in handy.h
docs the x-less version, adds 3 lines explaining the x-version.
I dont much care whether the 3 lines survive peer-review,
it looks like I can just truncate the patch and it will still apply.

Attachment: diff.newx.20050706.135719.bz2
Description: BZip2 compressed data

diff -ruN -X exclude-diffs newx/handy.h newxdoc/handy.h
--- newx/handy.h        2005-07-01 11:26:45.000000000 -0600
+++ newxdoc/handy.h     2005-07-01 13:18:38.000000000 -0600
@@ -534,17 +534,22 @@
 
 =head1 Memory Management
 
-=for apidoc Am|void|New|int id|void* ptr|int nitems|type
+=for apidoc Am|void|Newx|void* ptr|int nitems|type
 The XSUB-writer's interface to the C C<malloc> function.
 
-=for apidoc Am|void|Newc|int id|void* ptr|int nitems|type|cast
+=for apidoc Am|void|Newxc|void* ptr|int nitems|type|cast
 The XSUB-writer's interface to the C C<malloc> function, with
 cast.
 
-=for apidoc Am|void|Newz|int id|void* ptr|int nitems|type
+=for apidoc Am|void|Newxz|void* ptr|int nitems|type
 The XSUB-writer's interface to the C C<malloc> function.  The allocated
 memory is zeroed with C<memzero>.
 
+In 5.9.3, we removed the 1st parameter, a debug aid, from the api.  It
+was used to uniquely identify each usage of these allocation
+functions, but was deemed unnecessary with the availability of better
+memory tracking tools, valgrind for example.
+
 =for apidoc Am|void|Renew|void* ptr|int nitems|type
 The XSUB-writer's interface to the C C<realloc> function.
 

Reply via email to