Review points implementation

Author: lathapatil

bundles/org.eclipse.jface.text/src/org/eclipse/jface/text/SurroundWithBracketsStrategy.java

@@ -38,7 +38,7 @@ public SurroundWithBracketsStrategy(ISourceViewer sourceViewer) {
 	public void customizeDocumentCommand(IDocument document, DocumentCommand command) {
 		if (command.text != null && bracketsMap.containsKey(command.text)) {
 			try {
-				ITextSelection selection= (ITextSelection) sourceViewer.getSelectionProvider().getSelection();
+				ITextSelection selection= command.fSelection;
 				if (selection != null && selection.getLength() > 0) {
 					String selectedText= document.get(selection.getOffset(), selection.getLength());
 					String closingBracket= bracketsMap.get(command.text);

bundles/org.eclipse.jface.text/src/org/eclipse/jface/text/source/SourceViewerConfiguration.java

@@ -15,7 +15,6 @@
 package org.eclipse.jface.text.source;
 
 
-import java.util.ArrayList;
 import java.util.Arrays;
 
 import org.eclipse.swt.SWT;
@@ -192,11 +191,7 @@ public org.eclipse.jface.text.IAutoIndentStrategy getAutoIndentStrategy(ISourceV
 	 * @since 3.1
 	 */
 	public IAutoEditStrategy[] getAutoEditStrategies(ISourceViewer sourceViewer, String contentType) {
-
-		IAutoEditStrategy[] autoEditStrategies= new IAutoEditStrategy[] { getAutoIndentStrategy(sourceViewer, contentType) };
-		ArrayList<IAutoEditStrategy> autoEditStrategiesList= new ArrayList<>(Arrays.asList(autoEditStrategies));
-		autoEditStrategiesList.add(new SurroundWithBracketsStrategy(sourceViewer));
-		return autoEditStrategiesList.toArray(new IAutoEditStrategy[0]);
+		return new IAutoEditStrategy[] { getAutoIndentStrategy(sourceViewer, contentType), new SurroundWithBracketsStrategy(sourceViewer) };
 	}
 
 	/**