s3-talloc Change TALLOC_ZERO_P() to talloc_zero()

Stefan (metze) Metzmacher metze at samba.org
Thu Jun 23 12:52:35 MDT 2011

Am 23.06.2011 18:01, schrieb Jeremy Allison:
> On Thu, Jun 23, 2011 at 12:21:25PM +0200, Michael Adam wrote:
>> Andrew Bartlett wrote:
>>> On Fri, 2011-06-10 at 13:25 +0200, Michael Adam wrote:
>>> Rest assured I would never slip in 'fixup' changes unannounced into a
>>> commit such as this.
>> Yeah, I did not assume that, either. :-)
>>>> Don't worry about changin the code, please: I am just changing/fixing
>>>> this code file anyways. But I was just curious because I stumbled
>>>> over this conflict when I now tried to rebase changes I made... :-)
>>>> Again, my lesson is: Try to change as little files in one
>>>> commit as possible.
>>> There are still a number of other issues (types without _t, True and
>>> False) that come up every time I move code from source3 to the common
>>> code, so I am very keen to learn the lessons here:  How would you have
>>> handled this better?
> At some point I was planning to do a remove of "False", "True" and
> fix the compile of everything that breaks (converting to "false" and
> "true"). That would be a big change but a very targetted one. Is
> that something people would support ?

I'd say yes, but not in one single commit.

It's also possible to do one subdir per day or so,
that would also make possible conflicts easier to resolve for others.
And it makes cherry picking for backports much easier.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 262 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20110623/6edf5946/attachment.pgp>

More information about the samba-technical mailing list