Refactoring autocomplete structure

Author: ricardojlrufino

app/src/cc/arduino/autocomplete/BaseCCompletionProvider.java

@@ -0,0 +1,17 @@
+package cc.arduino.autocomplete;
+
+import org.fife.ui.autocomplete.DefaultCompletionProvider;
+
+/**
+ * Base completion provider for C/C++.
+ * @author Ricardo JL Rufino (ricardo@criativasoft.com.br)
+ * @date 28/04/2017
+ */
+public class BaseCCompletionProvider extends DefaultCompletionProvider{
+ 
+  @Override
+  protected boolean isValidChar(char ch) {
+    return super.isValidChar(ch) || '.' == ch || '>' == ch  || '-' == ch  || '<' == ch || '#' == ch || ':' == ch /**|| getParameterListStart() == ch */;
+  }
+
+}

app/src/cc/arduino/autocomplete/CompletionContext.java

@@ -3,20 +3,26 @@
 import java.util.LinkedList;
 import java.util.List;
 
+import javax.swing.text.JTextComponent;
+
 import org.fife.ui.autocomplete.BasicCompletion;
 import org.fife.ui.autocomplete.Completion;
 
-public class FakeCompletionProvider implements CompletionProvider {
-  
+import processing.app.syntax.SketchTextArea;
 
+public class FakeCompletionProvider extends BaseCCompletionProvider {
+  
   @Override
-  public List<Completion> getSuggestions(CompletionContext context) {
+  protected List<Completion> getCompletionsImpl(JTextComponent comp) {
     List<Completion> list = new LinkedList<>();
-    list.add(new BasicCompletion(context.getDelegate(), "Text: " + context.getAlreadyEnteredText()));
-    list.add(new BasicCompletion(context.getDelegate(), "Line: " + context.getLineNumber()));
+    
+    SketchTextArea area = (SketchTextArea) comp;
+    
+    list.add(new BasicCompletion(this, "Text: " + getAlreadyEnteredText(comp)));
+    list.add(new BasicCompletion(this, "Line: " + area.getCaretLineNumber()));
 
     return list;
   }
-  
+
 
 }

app/src/cc/arduino/autocomplete/CompletionProvider.java

@@ -1,33 +1,27 @@
-package cc.arduino.autocomplete.rsyntax;
+package cc.arduino.autocomplete;
 
+import org.fife.ui.autocomplete.CompletionProvider;
 import org.fife.ui.autocomplete.LanguageAwareCompletionProvider;
 
-import cc.arduino.autocomplete.CompletionContext;
-import cc.arduino.autocomplete.CompletionProvider;
 import processing.app.Sketch;
 import processing.app.syntax.SketchTextArea;
 
-
 /**
  * CompletionProvider for Arduino/CPP Language. <br/>
- * Setup basic logic for completions using {@link LanguageAwareCompletionProvider}
- * Filtering and decision will appear in the autocomplete dialog by implementations of: {@link CompletionProvider}. <br/>
+ * Setup basic logic for completions using {@link LanguageAwareCompletionProvider}. <br/>
+ * Filtering and decision will appear in the autocomplete dialog by implementations of: {@link CompletionProvider}.<br/>
  * 
  * @author Ricardo JL Rufino (ricardo@criativasoft.com.br)
  * @date 28/04/2017
  */
-public class SketchCompletionProvider extends LanguageAwareCompletionProvider{
-  
+public class SketchCompletionProvider extends LanguageAwareCompletionProvider {
+
   public SketchCompletionProvider(Sketch sketch, SketchTextArea textArea, CompletionProvider provider) {
 
-    CompletionContext context = new CompletionContext(sketch, textArea, this);
-    
-    setDefaultCompletionProvider(new CompletionProviderBridge(context, provider)); 
-//    provider.setParameterChoicesProvider(new ParameterChoicesProvider(this));
-//    provider.setParameterizedCompletionParams('(', ", ", ')');
-    
+    setDefaultCompletionProvider(provider);
+    // provider.setParameterChoicesProvider(new ParameterChoicesProvider(this));
+    // provider.setParameterizedCompletionParams('(', ", ", ')');
+
   }
-  
 
-  
 }

app/src/cc/arduino/autocomplete/FakeCompletionProvider.java

@@ -54,6 +54,7 @@
 
 import org.apache.commons.compress.utils.IOUtils;
 import org.fife.ui.autocomplete.AutoCompletion;
+import org.fife.ui.autocomplete.CompletionProvider;
 import org.fife.ui.rsyntaxtextarea.LinkGenerator;
 import org.fife.ui.rsyntaxtextarea.LinkGeneratorResult;
 import org.fife.ui.rsyntaxtextarea.RSyntaxDocument;
@@ -66,8 +67,7 @@
 import org.fife.ui.rtextarea.RTextArea;
 import org.fife.ui.rtextarea.RTextAreaUI;
 
-import cc.arduino.autocomplete.CompletionProvider;
-import cc.arduino.autocomplete.rsyntax.SketchCompletionProvider;
+import cc.arduino.autocomplete.SketchCompletionProvider;
 import processing.app.Base;
 import processing.app.BaseNoGui;
 import processing.app.PreferencesData;
@@ -86,8 +86,6 @@ public class SketchTextArea extends RSyntaxTextArea {
 
   private PdeKeywords pdeKeywords;
 
-  private SketchCompletionProvider completionProvider;
-
   public SketchTextArea(RSyntaxDocument document, PdeKeywords pdeKeywords) throws IOException {
     super(document);
     this.pdeKeywords = pdeKeywords;
@@ -102,9 +100,9 @@ public void setKeywords(PdeKeywords keywords) {
 
   public void setupAutoComplete(Sketch sketch, CompletionProvider provider) {
 
-    this.completionProvider = new SketchCompletionProvider(sketch, this, provider);
+    SketchCompletionProvider completionProvider = new SketchCompletionProvider(sketch, this, provider);
 
-    AutoCompletion ac = new AutoCompletion( this.completionProvider);
+    AutoCompletion ac = new AutoCompletion( completionProvider );
 
     ac.setAutoActivationEnabled(true);
     ac.setShowDescWindow(false);

app/src/cc/arduino/autocomplete/rsyntax/SketchCompletionProvider.java renamed to app/src/cc/arduino/autocomplete/SketchCompletionProvider.java

@@ -247,10 +247,6 @@ public boolean equals(Object o) {
     return (o instanceof SketchFile) && file.equals(((SketchFile) o).file);
   }
 
-  public Sketch getSketch() {
-	return sketch;
-  }
-
   /**
    * Load this piece of code from a file and return the contents. This
    * completely ignores any changes in the linked storage, if any, and