perl5 icon indicating copy to clipboard operation
perl5 copied to clipboard

ext/ modules - use AV _simple functions on new AVs

Open richardleach opened this issue 2 years ago • 1 comments

richardleach avatar Sep 09 '22 13:09 richardleach

I'll make similar changes to more modules under ext/ and add them to this PR. That will hopefully make reviewing more straightforward.

richardleach avatar Sep 17 '22 22:09 richardleach

I resolved conflicts on this.

demerphq avatar Feb 19 '23 14:02 demerphq

@richardleach, if you wish to pursue this pull request further, could you resolve merge conflicts and re-push? Thanks.

jkeenan avatar Nov 22 '23 22:11 jkeenan

@richardleach, because this (draft) p.r. touches many different files, it's going to be prone to merge conflicts. Should you wish to pursue it, could you resolve the current merge conflict(s) and reply to @Leont's question above? Thanks.

jkeenan avatar Dec 09 '23 13:12 jkeenan

@richardleach, because this (draft) p.r. touches many different files, it's going to be prone to merge conflicts. Should you wish to pursue it, could you resolve the current merge conflict(s) and reply to @Leont's question above? Thanks.

I'll get this done by early in Jan. Thanks.

richardleach avatar Dec 28 '23 17:12 richardleach

This has been rebased and Leon's macro suggestion implemented. Does anyone want to give this another once over before it gets merged?

richardleach avatar Jan 10 '24 21:01 richardleach

newAV_mortal() needs docs. "make test" currently outputs to stderr: no docs for newAV_mortal

iabyn avatar Jan 29 '24 10:01 iabyn

Thanks for the spot. I'll write something this week.

richardleach avatar Jan 29 '24 10:01 richardleach