On 9/11/20 4:22 AM, Jean-Marc Lasgouttes wrote:
> Hello,
>
> I get back to the list to continue the discussion.
>
> I like the patch. Did you manage to test it ? I do not know how to
> check that errors are caught.
>
> The following code
> +    bool valid = (pret == 0);
> +    if (!success)
> +        valid = false;
> should be replaced with
> +    bool valid = (pret == 0) && valid;

Which could then be const, I suspect.

Riki


-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to