Hi, On 13-12-16 20:54, Selva Nair wrote: > > On Tue, Dec 13, 2016 at 1:55 PM, Steffan Karger <stef...@karger.me > <mailto:stef...@karger.me>> wrote: > > > > > On 09/12/16 22:27, Steffan Karger wrote: > > > > > > Sounds like we have a final config on the CodeStyle page now. > Are we > > > ready to run it on all code now, and publish a reformat branch? > > > > > > > Agreed. I can do this later this night. > > > > Wait a minute we may need cmt_cpp_to_c=true -- I saw some C++ comments > > (added by uncrustify) in header files against #endif etc.. > > Indeed, tested and added to the CodeStyle page. Anyone up for creating > the reformatting branch? > > > I did some test over the weekend. For what its worth just pushed the branch > to https://github.com/selvanair/openvpn/tree/uncrustify > > This uses all options at the CodeStyle page plus > sp_bool=add
This makes sense. > sp_arith=add I think we agreed on IRC to not touch arithmetic spacing, because clarity really depends on the code. > pos_bool=lead What does this do exactly? The description "The position of boolean operators in wrapped expressions" from --show-config doesn't ring a bell for me. -Steffan ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
