Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1116)

Unified Diff: editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java

Issue 545023002: Generate setters for RefactoringOptions. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java
diff --git a/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java b/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java
index c45b022260606fcd3dbfec4777f1719a675271ea..6c3cbcd12c3f971af343b8e8aa5a4d7729fceafa 100644
--- a/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java
+++ b/editor/tools/plugins/com.google.dart.server/src/com/google/dart/server/generated/types/ExtractMethodOptions.java
@@ -44,18 +44,18 @@ public class ExtractMethodOptions extends RefactoringOptions {
/**
* The return type that should be defined for the method.
*/
- private final String returnType;
+ private String returnType;
/**
* True if a getter should be created rather than a method. It is an error if this field is true
* and the list of parameters is non-empty.
*/
- private final boolean createGetter;
+ private boolean createGetter;
/**
* The name that the method should be given.
*/
- private final String name;
+ private String name;
/**
* The parameters that should be defined for the method.
@@ -68,14 +68,14 @@ public class ExtractMethodOptions extends RefactoringOptions {
* - To add new parameters, omit their identifier.
* - To remove some parameters, omit them in this list.
*/
- private final List<RefactoringMethodParameter> parameters;
+ private List<RefactoringMethodParameter> parameters;
/**
* True if all occurrences of the expression or statements should be replaced by an invocation of
* the method. The expression or statements used to initiate the refactoring will always be
* replaced.
*/
- private final boolean extractAll;
+ private boolean extractAll;
/**
* Constructor for {@link ExtractMethodOptions}.
@@ -180,6 +180,52 @@ public class ExtractMethodOptions extends RefactoringOptions {
return builder.toHashCode();
}
+ /**
+ * True if a getter should be created rather than a method. It is an error if this field is true
+ * and the list of parameters is non-empty.
+ */
+ public void setCreateGetter(boolean createGetter) {
+ this.createGetter = createGetter;
+ }
+
+ /**
+ * True if all occurrences of the expression or statements should be replaced by an invocation of
+ * the method. The expression or statements used to initiate the refactoring will always be
+ * replaced.
+ */
+ public void setExtractAll(boolean extractAll) {
+ this.extractAll = extractAll;
+ }
+
+ /**
+ * The name that the method should be given.
+ */
+ public void setName(String name) {
+ this.name = name;
+ }
+
+ /**
+ * The parameters that should be defined for the method.
+ *
+ * It is an error if a REQUIRED or NAMED parameter follows a POSITIONAL parameter. It is an error
+ * if a REQUIRED or POSITIONAL parameter follows a NAMED parameter.
+ *
+ * - To change the order and/or update proposed paramerers, add parameters with the same
+ * identifiers as proposed.
+ * - To add new parameters, omit their identifier.
+ * - To remove some parameters, omit them in this list.
+ */
+ public void setParameters(List<RefactoringMethodParameter> parameters) {
+ this.parameters = parameters;
+ }
+
+ /**
+ * The return type that should be defined for the method.
+ */
+ public void setReturnType(String returnType) {
+ this.returnType = returnType;
+ }
+
public JsonObject toJson() {
JsonObject jsonObject = new JsonObject();
jsonObject.addProperty("returnType", returnType);

Powered by Google App Engine
This is Rietveld 408576698