Hi, On 08-12-16 23:24, Steffan Karger wrote: > On 08-12-16 22:56, Selva Nair wrote: >> Reformatted tun.c has at least two instances of char* xxx, we want char >> *xxx, right? So why not add some sp_xxx_ptr_star options. > > We settled for char *xxx yesterday, so I'll add that. > >> Less of a concern at least for me is function_name (...) and >> function_name(...) both appears to be allowed. Just because we can do >> it, why not add some sp_func_xxxx_paren options to normalize those. > > Agreed. Allman (at least according to the wikipedia examples) uses > funcion_name(), so I'll add that too. > > I'll update the options on the CodeStyle page.
Done, new tun.c example here: https://paste.fedoraproject.org/502063/36953148/ -Steffan ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today.http://sdm.link/xeonphi _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
