Remaining "cast"/"const" patches

Jelmer Vernooij jelmer at
Sun Oct 18 11:27:15 MDT 2009

Hi Matthias,

On Sun, 2009-10-18 at 16:25 +0000, Matthias Dieter Wallnöfer wrote:
> I finished the split-up. It's not reasonable to post the patches as attachment - so they're only in my private repo. Maybe you like to write the filenames or patch titles of those which look okay.
Do you have a link to your private repository handy? It doesn't seem to
be on at least...


> Matthias
> --- Matthias Dieter Wallnöfer <mdw at> schrieb am Fr, 16.10.2009:
> Von: Matthias Dieter Wallnöfer <mdw at>
> Betreff: Re: Remaining "cast"/"const" patches
> An: "Andrew Bartlett" <abartlet at>
> CC: "Samba Technical Mailinglist" <samba-technical at>
> Datum: Freitag, 16. Oktober 2009, 12:32
> Andrew,
> well - okay - I'll definitely split them up. I see it's too much work in one commit.
> Matthias
> --- Andrew Bartlett <abartlet at> schrieb am Fr, 16.10.2009:
> Matthias,
> For as long as you continue to post these as large 'fix things' patches,
> we will have to continue to reject them.  I'm sorry it has to be this
> way, and I know it is a *lot* of work to split up the patches, but it is
> even more work to track down problems later, and Volker is right to be
> concerned when there are behaviour changes mixed in with casts. 
> You even just have other stylistic changes mixed in with these patches.
> A change such as the one to nbtname might be correct, but it is also a
> great example of why it needs to be separated, so it can be clearly
> explained in it's own commit message, and clearly understood independent
> from other changes.  This same need applies to almost every change in
> this series.  
> Perhaps you don't know how to use 'git add -i' to create small patches
> from one file?  (A git guru on #samba-technical will no doubt be glad to
> give you a tutorial). 
> Andrew Bartlett

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <>

More information about the samba-technical mailing list