in reply to Re: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc
in thread Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc
I have posted it (well, an earlier hackier version) to p5p. I'm looking more for things like: mayberelocate() is an awful function name. that's not how you use SvREFCNT_dec(). you should put your function above S_incpush_if_exists(). needs more comments.
I'd like to improve my chances of my patch being accepted before submitting it via perlbug. It might get overlooked or just passed over for being too crude on p5p, and well, the more looking over it gets the better.
|
|---|
| Replies are listed 'Best First'. | |
|---|---|
|
Re^3: Rate my patch: fix -Dusesitecustomize + -Duserelocatableinc
by Tux (Canon) on Nov 18, 2011 at 13:34 UTC |