[PART4] [PATCHES] generate the param_table (FINALLY)

Michael Adam obnox at samba.org
Wed Jul 29 10:20:43 UTC 2015


Hi,

FYI: I have rebased the branch mentioned below.
The code still cleanly applies and builds...

:-)

Cheers - Michael


On 2015-07-24 at 02:21 +0200, Michael Adam wrote:
> The complete patchset is also available in my repository:
> 
> git://git.samba.org/obnox/samba/samba-obnox.git
> 
> branch master-loadparm-ok
> 
> gitweb:
> 
> https://git.samba.org/?p=obnox/samba/samba-obnox.git;a=shortlog;h=refs/heads/master-loadparm-ok
> 
> Cheers - Michael
> 
> On 2015-07-24 at 02:13 +0200, Michael Adam wrote:
> > So here is finally the last patchset in this series.
> > 
> > It removes our committed param_table and has it
> > generated from the docs.
> > 
> > I have arranged the patches in such a way that
> > an intermediate state proves that the generated table
> > is correct before removing the static one:
> > 
> > - split out the committed struct into param_table_static.c
> >   and include it in param_table.c
> > - add the autogeneration to param_table_gen.c
> > - add a test to verify that the tables are identical
> >   (modulo order)
> > - switch the include to use the generated table
> > - remove all tests that use the static table
> > - remove the generated table
> > - update documentation.
> > 
> > One thing that I might imagine autogeneration
> > in the future is setting of the default values.
> > This is now the only part that is not generated
> > from the docs at this point. But usually, we now
> > only need to touch one xml file to add or change
> > a parameter. :-)
> > 
> > Review appreciated.
> > 
> > Cheers - Michael
> > 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150729/cbd3f0b9/attachment.sig>


More information about the samba-technical mailing list