On Saturday 07 January 2006 18:39, Philip M. Gollucci wrote: > > As a more generic solution, this might make sense. On first glance, your > > patch looks sane, but I won't have the time to really look at it until > > Jan 15th, so if anybody else feels like writing a few tests for this and > > beat me to it... > > Wouldn't that be an API change ?
I would say an API extension. add_config is prototyped to expect up to 3 parameters $r, $lines and $override. If called with more it throws an exception. Hence, if a new 4th optional parameter is added it cannot break any existing code. Torsten
pgpL1h5nyYlu3.pgp
Description: PGP signature