You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by jo...@apache.org on 2010/12/03 11:55:17 UTC

svn commit: r1041771 - /uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java

Author: joern
Date: Fri Dec  3 10:55:17 2010
New Revision: 1041771

URL: http://svn.apache.org/viewvc?rev=1041771&view=rev
Log:
UIMA-1875 Added a comment about the performance of the box drawing strategy call

Modified:
    uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java

Modified: uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java
URL: http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java?rev=1041771&r1=1041770&r2=1041771&view=diff
==============================================================================
--- uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java (original)
+++ uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/annotation/TagDrawingStrategy.java Fri Dec  3 10:55:17 2010
@@ -62,6 +62,11 @@ class TagDrawingStrategy implements IDra
           Color color) {
     
     // Always draw a box around the annotation itself
+    
+    // TODO: It seems that the call to the box drawing strategy is rather
+    // expensive, test how fast it is when the box drawing is "inlined".
+    // The box drawing strategy could be changed to do the drawing via
+    // static methods
     annotationDrawingStyle.draw(annotation, gc, textWidget, offset, length, color);
     
     if (annotation instanceof EclipseAnnotationPeer) {