[PATCH] Generate param_functions.c and associated files from XML documentation
Michael Adam
obnox at samba.org
Tue Feb 11 16:50:00 MST 2014
Hi Andrew,
On 2014-02-12 at 10:20 +1300, Andrew Bartlett wrote:
>
> This is the current patch series for the loadparm work Garming and I
> have done. I'm sorry to have to note that it has now been 4 weeks since
> this was first posted on 14 Jan.
Yeah, sorry, but big patchsets take time, and
there are subtleties hidden here that did not advise
rushing this in...
I have reviewed the patches and added reviewed-by-/me tags to
most of them in my master-param branch.
There are two questions/remarks:
1) I think we need to special case 0 for smb2 max credits
as a protection: if we leave the value at 0,
we do "... % 0" in the code using this...
So the patch as such is not acceptable.
2) could we split the patch
"lib/param: Deprecate and normalise "read raw" and "write raw"
parameters"?
we should not do normalization and deprecation in one patch,
imho.
Also: why do you deprecate them at all?
If you remove the deprecation (doc change)
and only do the normalization for a start,
you have my reviewed-by for it.
So skipping the max credits patch for a start
and skipping the deprecation for the read/write raw
patch, feel free to push w/ my review.
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 215 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140212/ea858967/attachment.pgp>
More information about the samba-technical
mailing list