Tk Source Code

The following patch prevents the assertion from triggering in the situation I can otherwise make it trigger repeatably in my application:

<verbatim>
Index: generic/tkText.c
==================================================================
--- generic/tkText.c
+++ generic/tkText.c
@@ -1865,11 +1865,11 @@
                break;
            case 'y':
                if (strncmp("-ypixels", option, length) == 0) {
                    int from, to;

-                   if (update) {
+                   if (update && (TkTextIndexCompare(&indexFrom, &indexTo) != 0)) {
                        from = TkTextIndexGetLineNumber(&indexFrom, textPtr);
                        to = TkTextIndexGetLineNumber(&indexTo, textPtr);
                        UpdateLineMetrics(textPtr, from, to);
                    }
                    from = TkTextIndexYPixels(textPtr, &indexFrom);
</verbatim>

Is this correct, or is there a better fix?