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

Unified Diff: pkg/analysis_server/doc/api.html

Issue 2312073003: Issue 27078. Describe RefactoringProblemSeverity values. (Closed)
Patch Set: Created 4 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
« no previous file with comments | « no previous file | pkg/analysis_server/lib/plugin/protocol/generated_protocol.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analysis_server/doc/api.html
diff --git a/pkg/analysis_server/doc/api.html b/pkg/analysis_server/doc/api.html
index 80b20f3bbe0af30ae5ecbd7ab1daee1ee369ebb0..6637ad7b3d778d026adf37d85caeccf253f356c0 100644
--- a/pkg/analysis_server/doc/api.html
+++ b/pkg/analysis_server/doc/api.html
@@ -3674,7 +3674,37 @@ a:focus, a:hover {
returned by the refactoring requests.
</p>
- <dl><dt class="value">INFO</dt><dt class="value">WARNING</dt><dt class="value">ERROR</dt><dt class="value">FATAL</dt></dl></dd><dt class="typeDefinition"><a name="type_RemoveContentOverlay">RemoveContentOverlay: object</a></dt><dd>
+ <dl><dt class="value">INFO</dt><dd>
+
+ <p>A minor code problem. No example, because it is not used yet.
+ </p>
+ </dd><dt class="value">WARNING</dt><dd>
+
+ <p>A minor code problem. For example names of local variables
+ should be camel case and start with a lower case letter. Staring
+ the name of a variable with an upper case is OK from the language
+ point of view, but it is nice to warn the user.
+ </p>
+ </dd><dt class="value">ERROR</dt><dd>
+
+ <p>The refactoring technically can be performed, but there is a
+ logical problem. For example the name of a local variable being
+ extracted conflicts with another name in the scope, or
+ duplicate parameter names in the method being extracted, or
+ a conflict between a parameter name and a local variable, etc.
+ In some cases the location of the problem is also provided, so
+ the IDE can show user the location and the problem, and let the
+ user decide whether she wants to perform the refactoring. For
+ example the name conflict might be expected, and the user wants
+ to fix it afterwards.
+ </p>
+ </dd><dt class="value">FATAL</dt><dd>
+
+ <p>A fatal error, which prevents performing the refactoring.
+ For example the name of a local variable being extracted is not a
+ valid identifier, or selection is not a valid expression.
+ </p>
+ </dd></dl></dd><dt class="typeDefinition"><a name="type_RemoveContentOverlay">RemoveContentOverlay: object</a></dt><dd>
<p>
A directive to remove an existing file content overlay.
After processing this directive, the file contents will once
« no previous file with comments | « no previous file | pkg/analysis_server/lib/plugin/protocol/generated_protocol.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698