I won't be able to have a serious look until Monday, as I'm still remote.
Hopefully you arent working on weekends :)

On Fri, Sep 25, 2015 at 6:30 PM Todd Fiala <todd.fi...@gmail.com> wrote:

> tfiala added inline comments.
>
> ================
> Comment at: test/dosep.py:245
> @@ +244,3 @@
> +        # binary should have called the results-generation code.
> +        raise Exception("no test results were generated whatsoever")
> +    return process_driver.results
> ----------------
> This message should include the filename.
>
> ================
> Comment at: test/process_control.py:273
> @@ +272,3 @@
> +
> +    def create_piped_process(self, command, new_process_group=True):
> +        if new_process_group:
> ----------------
> This will want to store the popen_process.using_process_groups property
> like I do on the Posix side if the terminate would do something different
> based on the presence or absence of using process groups.  Oversight for
> not storing.
>
>
> http://reviews.llvm.org/D13124
>
>
>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to