[PR PATCH] [Updated] Added smbc_SetLogCallback which lets third party code to capture libsmbclient logs
Andrew Bartlett
abartlet at samba.org
Mon Dec 4 09:38:15 UTC 2017
On Mon, 2017-12-04 at 14:29 +0530, Puran Chand wrote:
> Hi Andrew,
>
> Added code in libsmbclient.c to verify the API.
> Please review.
Thanks. I'm not quite sure what you are trying to do in
debug_callback(), you seem to be putting the test code in there, rather
than in the test routine and just seeing if you got a flag back (via
the private pointer). It should be just after the DEBUG() line.
Also, on the buffering, you just need a \n and it will flush the
buffer, no need to fill it.
You are on the right track. For the final submission check
README.Coding and so avoid \\ comments.
Thanks for giving this a go!
Andrew Bartlett
> On Mon, Dec 4, 2017 at 12:54 PM, Github bot account via samba-technical <samba-technical at lists.samba.org> wrote:
> > There is an updated pull request by puran157 against master on the Samba Samba Github repository
> >
> > https://github.com/puran157/samba set_log_callback_libsmbclient
> > https://github.com/samba-team/samba/pull/112
> >
> > Added smbc_SetLogCallback which lets third party code to capture libsmbclient logs
> >
> >
> > A patch file from https://github.com/samba-team/samba/pull/112.patch is attached
> >
>
>
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list