Patches to review

Richard Kimberly Heck rikiheck at lyx.org
Wed Dec 2 17:01:15 UTC 2020


On 12/2/20 10:32 AM, Yuriy Skalko wrote:
> Next refactoring patches.

#1 and #3 are fine. I'll leave #4 again to Pavel and others.

In #2, we have:

>      /// who initiated the action
> -    Origin origin_;
> +    Origin origin_ = INTERNAL;
It doesn't look as if there was a default before.

I often wonder if there is some mechanism through which to guarantee 
that the constructor explicitly initializes a member, such as this one. 
Is there? (Otherwise, this is also fine.)

Riki




More information about the lyx-devel mailing list