[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [1929] trunk/org.rubypeople.rdt.core/src/org/ rubypeople/rdt/internal/codeassist/CompletionContext.java

[<caw...@us...>]

Revision: 1929
          http://svn.sourceforge.net/rubyeclipse/?rev=1929&view=rev
Author:   cawilliams
Date:     2007-02-07 07:24:15 -0800 (Wed, 07 Feb 2007)

Log Message:
-----------


Modified Paths:
--------------
    trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/internal/codeassist/CompletionContext.java

Modified: trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/internal/codeassist/CompletionContext.java
===================================================================
--- trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/internal/codeassist/CompletionContext.java	2007-02-07 15:23:15 UTC (rev 1928)
+++ trunk/org.rubypeople.rdt.core/src/org/rubypeople/rdt/internal/codeassist/CompletionContext.java	2007-02-07 15:24:15 UTC (rev 1929)
@@ -27,9 +27,6 @@
 		// Read from offset back until we hit a: space, period
 		// if we hit a period, use character before period as offset for
 		// inferrer
-		// if we hit a space, use character after space?
-		// TODO We need to handle other bad syntax like invoking completion
-		// right after an @
 		StringBuffer tmpPrefix = new StringBuffer();
 		for (int i = offset; i >= 0; i--) {
 			char curChar = source.charAt(i);
@@ -38,6 +35,7 @@
 				case '.': // if it breaks syntax, lets fix it
 				case '$':
 				case '@':
+					// TODO What if there is a valid character after this, so syntax isn't broken?
 					source.deleteCharAt(i);
 					break;
 				}


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.