[EMAIL PROTECTED] wrote:
-            if (ignoreEmptyTokens) {
+            if (this.isIgnoreEmptyTokens()) {
                 return;
             }
-            if (emptyAsNull) {
+            if (this.isEmptyTokenAsNull()) {
                 tok = null;
             }

While I understand most of the change here, with the use of methods rather than direct field access, I dislike the above.

The 'this.' just seriously gets in the way of reading the code and adds nothing of value. Perhaps it is just an artifact of a tool you used?

Stephen

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to