Re: [lmi] Not urgent: a plausible micro-optimization

[Greg Chicares]

On 2018-02-17 16:56, Vadim Zeitlin wrote:
[...]
>       https://github.com/vadz/lmi/pull/68


I have a question about one detail here:

+        wxDocMDIChildFrame const* c = dynamic_cast<wxDocMDIChildFrame*>(w);
         if(c)
             {
             IllustrationView* v = 
dynamic_cast<IllustrationView*>(c->GetView());
             if(v)
                 {
                 v->DisplaySelectedValuesAsHtml();
+
+                // Update the display immediately to display the new data as
+                // soon as possible, without waiting for the other views.
+                v->GetFrame()->Update();
                 }
             }

On the last added line, would
  c->Update();
do the same thing as
  v->GetFrame()->Update();
because 'c' is certainly the wxDocMDIChildFrame of 'v'?
Is it necessarily the one-and-only frame window of 'v'?

I've temporarily added this for testing:
+                LMI_ASSERT(v->GetFrame() == c);
                 v->GetFrame()->Update();
and I haven't found a way to make that assertion fail, but
I'd like to be sure I'm not missing some subtlety.