Removing the original params.c parser

Christopher R. Hertel crh at
Fri Aug 15 10:12:25 MDT 2014

On 08/15/2014 10:51 AM, Jeremy Allison wrote:
> On Fri, Aug 15, 2014 at 05:29:30PM +0200, Michael Adam wrote:
>> Chris,
>> On 2014-08-15 at 10:00 -0500, Christopher R. Hertel wrote:
>>> On 08/15/2014 04:19 AM, Andrew Bartlett wrote:
>>> :
>>>> Chris:  I know this is particularly difficult for you, but even for code
>>>> that needs as little attention as this, it is important that it be
>>>> consistent style-wise with the rest of the codebase.  This applies even
>>>> in this case, where it came before these conventions and so much else in
>>>> Samba.  
>>> This isn't even the slightest bit difficult for me.  It's old code that has
>>> served us well for a very long time.  I'm completely proud of my
>>> contributions here and completely happy to have it all ripped out and
>>> replaced by something that meets the current needs of the project.
>>> ...and you didn't actually read my original comments.  I'm recommending all
>>> or nothing, and favoring "all" where "all" is a complete conversion to the
>>> currently preferred formatting style.  If "all" means replacing the code
>>> completely that's fine too.
>> Thanks for clarifying this.
>> (It was clear to me... ;-)
>> Since the use of the new tini parser has already been brought
>> upstream, and you have expressed your non-concerns, would you mind
>> pushing Volker's "remove unused code" patch (or giving your review),
>> so as to eliminate any remaining doubts, and thereby finally
>> settling this exasperating discussion? :-)
>> I would greatly appreciate this!
> Well Chris already gave his +1, which goes as a
> "Reviewed-by:" for me so I'm inclined to just push it :-).
> Might update the description to mention what replaces
> it, but that's about it :-).
> Thanks Chris !

To be honest, I haven't actually reviewed the code itself, just the
discussion of the discussion of the patch.  :) :) :)

"Chaos, panic, and work here is done."  No further
interference from my corner.  Go for it, Jeremy.

Chris -)-----

"Implementing CIFS - the Common Internet FileSystem" ISBN: 013047116X
Samba Team --     -)-----   Christopher R. Hertel
jCIFS Team --   -)-----   ubiqx development, uninq.
ubiqx Team --     -)-----   crh at
OnLineBook --    -)-----   crh at

More information about the samba-technical mailing list