On Mon, Apr 14, 2008 at 7:43 PM, Jani Taskinen <[EMAIL PROTECTED]> wrote:
> Hannes Magnusson kirjoitti:
> > On Mon, Apr 14, 2008 at 5:40 PM, Jani Taskinen <[EMAIL PROTECTED]>
> wrote:
> >
> > > Eh..do you really understand the code correctly..? ini_get_all() is very
> > > much different what config_dump_hash() was/is.
> > >
> >
> > Uhh.. I don't see the difference. And there are no tests to compare it
> against.
> > I already asked on internals@ and after no reply I tracked down helly
> > on IRC and he was fine with merging them.
> >
>  This function was originally just meant for debugging the config hash.
>  And hint for the difference is: registered vs. anything (that was
> freebie..)
>  Rule 1: Learn how to use CVS before you commit (hint: cvs annotate
> filename.ext)
>  Rule 2: Once you learned how to use CVS and figured out who to ask: ASK.
>  Rule 3: Ask the right people before committing.
>  Rule 4: IRC != internals@ (I haven't seen ANY question about this on
> internals!)
>  Now be good boy and revert the crap, that function is VERY useful for
> debugging how the php.ini stuff works (or not) and should be there. It was
> originally only enabled for debug builds and should always still be, I've no
> idea why Marcus made it enabled always in the first place..

Rule#1: Test the stuff you write, not only in your mind, with .phpt too
Rule#2: Make sure there are not more than 3 functions doing almost the
exact same thing.
Rule#3: Subscribe to internals@ and participate in the discussion of your work
Rule#4: Add comments about any hidden meanings behind the function
Rule#5: Add a frckin [DOC] note to the commit explaining how the stuff
works so it can be documented
Rule#6: If you write a function just to so it can be used when
debugging another internal feature add it in #ifdef FEATURE_DEBUG or
whatever so other people don't come after you and wonder wtf you were
Rule#7: If a documentor asks for clarifications on internals@ then help him
Rule#8: Do not reclassify bug-reports as documentation problem without
further information

If you don't tell anyone about what you are doing you cannot expect
anyone else to know what your intentions are. If the function should
be documented, removed, changed, only in debug mode, only there or
only here. Marcus apparently thought you made a booboo and renamed it
and enabled it in normal builds, and I seeing absolutely no point in
yet another ini-get function combined it with another function
seemingly serving the same purpose, in slightly different format...

I did ask on internals@ if we really did need yet-another-ini-get
function and since noone replied I followed your basic principle:
Commit and let the trolls flame.

(learned from the best! ;))

PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to