Remaining "cast"/"const" patches

Matthias Dieter Wallnöfer mdw at samba.org
Fri Oct 16 04:32:03 MDT 2009


Andrew,

well - okay - I'll definitely split them up. I see it's too much work in one commit.

Matthias

--- Andrew Bartlett <abartlet at samba.org> 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

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Cisco Inc.



      


More information about the samba-technical mailing list