I believe that code is very, very old. At the time, I'm sure there was a reason 
for it, but we obviously don't need it now.  :-)

On Sep 15, 2010, at 9:44 PM, Todd Volkert wrote:

> Curious.  Makes me wonder why they were originally repainted with the
> immediate flag set to true...
> 
> -T
> 
> On Mon, Sep 13, 2010 at 8:13 PM,  <[email protected]> wrote:
>> Author: gbrown
>> Date: Tue Sep 14 00:13:22 2010
>> New Revision: 996726
>> 
>> URL: http://svn.apache.org/viewvc?rev=996726&view=rev
>> Log:
>> Resolve PIVOT-630.
>> 
>> Modified:
>>    
>> pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTextInputSkin.java
>>    pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextAreaSkin.java
>>    pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextPaneSkin.java
>> 
>> Modified: 
>> pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTextInputSkin.java
>> URL: 
>> http://svn.apache.org/viewvc/pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTextInputSkin.java?rev=996726&r1=996725&r2=996726&view=diff
>> ==============================================================================
>> --- 
>> pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTextInputSkin.java
>>  (original)
>> +++ 
>> pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraTextInputSkin.java
>>  Tue Sep 14 00:13:22 2010
>> @@ -66,7 +66,7 @@ public class TerraTextInputSkin extends
>> 
>>             if (caret != null) {
>>                 TextInput textInput = (TextInput)getComponent();
>> -                textInput.repaint(caret.x, caret.y, caret.width, 
>> caret.height, true);
>> +                textInput.repaint(caret.x, caret.y, caret.width, 
>> caret.height);
>>             }
>>         }
>>     }
>> 
>> Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextAreaSkin.java
>> URL: 
>> http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextAreaSkin.java?rev=996726&r1=996725&r2=996726&view=diff
>> ==============================================================================
>> --- pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextAreaSkin.java 
>> (original)
>> +++ pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextAreaSkin.java Tue Sep 
>> 14 00:13:22 2010
>> @@ -56,7 +56,7 @@ public class TextAreaSkin extends Compon
>> 
>>             if (selection == null) {
>>                 TextArea textArea = (TextArea) getComponent();
>> -                textArea.repaint(caret.x, caret.y, caret.width, 
>> caret.height, true);
>> +                textArea.repaint(caret.x, caret.y, caret.width, 
>> caret.height);
>>             }
>>         }
>>     }
>> 
>> Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextPaneSkin.java
>> URL: 
>> http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextPaneSkin.java?rev=996726&r1=996725&r2=996726&view=diff
>> ==============================================================================
>> --- pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextPaneSkin.java 
>> (original)
>> +++ pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TextPaneSkin.java Tue Sep 
>> 14 00:13:22 2010
>> @@ -62,7 +62,7 @@ public class TextPaneSkin extends Contai
>> 
>>             if (selection == null) {
>>                 TextPane textPane = (TextPane)getComponent();
>> -                textPane.repaint(caret.x, caret.y, caret.width, 
>> caret.height, true);
>> +                textPane.repaint(caret.x, caret.y, caret.width, 
>> caret.height);
>>             }
>>         }
>>     }
>> 
>> 
>> 

Reply via email to