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


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

Reply via email to