[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Mon Feb 28 14:56:13 MST 2011


On Mon, Feb 28, 2011 at 10:51:27PM +0100, Volker Lendecke wrote:

> Jeremy, I think this is the wrong fix. I have sent a patch
> to Tridge CC samba-technical that fixes it in a different
> way. Can you please coordinate with Tridge about the proper
> way to fix this? I did not push it directly with the big
> stream because this touches a core definition of tevent,
> which needs proper approval by Tridge.

Well I just fixed the warnings on a 64-bit box. It's not
a core change.

> Despite it might seem that I am neglecting my code, I'm
> trying my best to take care of it and get it into proper
> shape.
> 
> What is your argument against my fix with the union? Is that
> not portable enough? Is it too much bloat?

I didn't see your message before my autobuild got sent :-).

That's the simple answer. I like your fix better. Please
feel free to revert and push yours instead :-).

Jeremy.


More information about the samba-technical mailing list