[PATCH] Fix 2 clang warnings
Volker Lendecke
Volker.Lendecke at SerNet.DE
Mon Nov 24 08:14:51 MST 2014
On Mon, Nov 24, 2014 at 10:08:47AM -0500, Ira Cooper wrote:
> - tmpel = tmpel;
> }
> tmpel->values[tmpel->num_values -1].data
> =talloc_steal(tmpel->values, el->values[k].data);
> tmpel->values[tmpel->num_values -1].length =
> el->values[k].length;
> --
> 1.9.1
>
> I'm a bit concerned about this one. Matthieu, can you shed some light on
> this line? (I think the git blame pointed the finger at you...)
>
> It really looks like it could be a compiler bug workaround or something
> bizarre like that that?
>
> Otherwise, it's clearly "right". But lines like that.... scare me a bit.
Yes, same feeling here. But this is old, and the bug might
be gone. If we need it, we should clearly mark it with a
comment.
Volker
--
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de
More information about the samba-technical
mailing list