Re: [PATCH] Move "tee" building down to interpreter::set_logging_proc

Simon Marchi
2017-02-02T15:17:43Z

On 2017-02-02 09:28, Pedro Alves wrote:
> @@ -109,13 +109,13 @@ extern int current_interp_named_p (const char
> *name);
>
> /* Call this function to give the current interpreter an opportunity
> to do any special handling of streams when logging is enabled or
> - disabled. START_LOG is 1 when logging is starting, 0 when it ends,
> - and OUT is the stream for the log file; it will be NULL when
> - logging is ending. LOGFILE is non-NULL if the output streams
> + disabled. START_LOG is true when logging is starting, false when

START_LOG is not there anymore. From what I understand, it's replaced
with LOGFILE being null or not?

Otherwise, LGTM.