[PATCH] Some fixes for Solaris CC

Volker Lendecke Volker.Lendecke at SerNet.DE
Wed Jan 30 20:53:31 UTC 2019


On Wed, Jan 30, 2019 at 09:23:43PM +0100, swen via samba-technical wrote:
> The inner union only has one "member", 
> is it not easier to just remove the union ?
> 

This is something that the authors of this could should comment on. I
am not the designer of this, and I wanted the minimum necessary change
to get this in.

> In addition, you could fix the 80 column violation of the lines you're
> modifying.... ok with the above change not anymore :-)

Ok, you're right. Normally I modify the 80-col violations in this kind
of patch. This time I opted for the minimum necessary change. I can
re-submit if that's a problem for you.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: 0551-370000-0, mailto:kontakt at sernet.de
Gesch.F.: Dr. Johannes Loxen und Reinhild Jung
AG Göttingen: HR-B 2816 - http://www.sernet.de



More information about the samba-technical mailing list