CGContextRestoreGState: invalid context

[szabgab]

In #159 Chris Dolan mentioned

Exactly once per second, I get a new warning message on the console that says "CGContextRestoreGState: invalid context".

On Mac, Wx Version: 0.88 wxWidgets 2.8.8 using r1449

It seems this is related to this: ​http://trac.wxwidgets.org/ticket/4272

[schwern]

Me too. Except it doesn't seem to be once per second but once per keystroke.

Wx Version: 0.89 wxWidgets 2.8.9

This is perl, v5.10.0 built for darwin-thread-multi-2level

Padre 0.27

[azawawi]

Is this bug still valid with Padre 0.46 ?
Can you test it?

[schwern]

Still a problem in 0.39. Let's install 0.46 and see...

I'm having difficulty getting Wx 0.91 or 0.92 to fully compile.

[szabgab]

We are at 0.84 and have not heard about this issue recently. Could please check if it's still there?

[schwern]

Sorry, I still can't compile Wx on OS X. See ​https://rt.cpan.org/Ticket/Display.html?id=66723

[tome]

I can confirm this issue is still there on osx perl 5.14.1, Padre 0.87 (trunk r14962 Wx : .9901, wxw: 2.8.8)

<Error>: CGContextRestoreGState: invalid context 0x0

[tome]

I can confirm this is occurring because of updates to the status bar during the EVT_STC_UPDATEUI, as is indicated in the widgets ticket: ​http://trac.wxwidgets.org/ticket/4272

For example if I move the update code out of the UPDATEUI and into an idle timer, we don't get that error.

Wx::Event::EVT_STC_UPDATEUI( $self, -1, sub {

shift->{_do_update_ui} = 1;

} );

Wx::Event::EVT_IDLE($self, sub {

my $self = shift;
if($self->{_do_update_ui}) {

$self->{_do_update_ui} = undef;
$self->on_stc_update_ui();

}

});

This is all new code to me but at least it's a start.

[bowtie]

tome diff for above

lib/Padre/Wx/Main.pm

@@ -288 +288 @@
     );
 
     # Scintilla Event Hooks
-    Wx::Event::EVT_STC_UPDATEUI( $self, -1, \&on_stc_update_ui );
+    Wx::Event::EVT_STC_UPDATEUI(
+        $self, -1,
+        sub {
+            shift->{_do_update_ui} = 1;
+        }
+    );
+
+    Wx::Event::EVT_IDLE(
+        $self,
+        sub {
+            my $self = shift;
+            if ( $self->{_do_update_ui} ) {
+
+                $self->{_do_update_ui} = undef;
+                $self->on_stc_update_ui();
+            }
+        }
+    );
+
+    # Wx::Event::EVT_STC_UPDATEUI( $self, -1, \&on_stc_update_ui );
     Wx::Event::EVT_STC_CHANGE( $self, -1, \&on_stc_change );
     Wx::Event::EVT_STC_STYLENEEDED( $self, -1, \&on_stc_style_needed );
     Wx::Event::EVT_STC_CHARADDED( $self, -1, \&on_stc_char_added );

[tome]

Patch to defer on_stc_update_ui to the idle after the event

[tome]

I'm also now seeing these context warnings when doing fast swipe scrolling, so I'll need to figure out where that's coming from and if a more general solution should be implemented.

[tome]

With r15298, there is now only one known case of this warning:
When using the default scintilla, we get this when scrolling to new lines.

After debugging and eliminating all the Padre calls, I believe this may be in the Wx's implementation. I'm hoping to find out more once I can, or when I get to chat with those folks again.

For now this bug is 1/2 fixed, but at least we don't see the warning on every single keystroke.

[azawawi]

Close it? :)