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

Issue 10828389: Enhance 'Create Method' Quick Fix. (Closed)

Created:
8 years, 4 months ago by scheglov
Modified:
8 years, 4 months ago
Reviewers:
messick
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Enhance 'Create Method' Quick Fix. 1. Support for creating methods in classes, not just top-level functions. 2. Show suggestions for parameter names. 3. Show suggestions for parameter types. R=messick@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=10988

Patch Set 1 #

Total comments: 2

Messages

Total messages: 3 (0 generated)
scheglov
8 years, 4 months ago (2012-08-20 15:53:21 UTC) #1
messick
lgtm https://chromiumcodereview.appspot.com/10828389/diff/1/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java File editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java (right): https://chromiumcodereview.appspot.com/10828389/diff/1/editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java#newcode208 editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java:208: private void addFix_createUnresolvedMethod(IProblemLocation location) throws Exception { Please ...
8 years, 4 months ago (2012-08-20 16:48:13 UTC) #2
scheglov
8 years, 4 months ago (2012-08-20 17:02:01 UTC) #3
https://chromiumcodereview.appspot.com/10828389/diff/1/editor/tools/plugins/c...
File
editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java
(right):

https://chromiumcodereview.appspot.com/10828389/diff/1/editor/tools/plugins/c...
editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/text/correction/QuickFixProcessor.java:208:
private void addFix_createUnresolvedMethod(IProblemLocation location) throws
Exception {
On 2012/08/20 16:48:13, messick wrote:
> Please justify throwing Exception in javadoc, or change to more specific
> exception class(es).

Specifying general Exception class was deliberate decision.
We don't have here any way to recover from exception, so no reason to specify
them.
Also, this is just small UI feature, at worst case you will press Cmd+1 and it
will do nothing, fail silently. It will not show any scary dialog to user.

Powered by Google App Engine
This is Rietveld 408576698