On Wed, 2008-01-30 at 16:04 -0800, Wayne Davison wrote:
> On Tue, Jan 29, 2008 at 09:45:08PM -0500, Matt McCutchen wrote:
> > Since the process executing the code inside the previous "if" (which
> > doesn't close the log file) sets am_server = 1, I think the comment is
> > backwards.
> 
> No, it's saying (or attempting to say) that the client will handle the
> logging, so the server is closing the logfile.

I understand what the comment is currently attempting to say.  My point
is that, unless I am misreading the code, the log-closing code is
reached by the client and not by the server (which enters the body of
the "if (pid == 0)" block, sets am_server = 1, and calls child_main,
which presumably exits the process without returning).  Since the client
closes the log, the server is doing the logging.

Matt

-- 
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

Reply via email to