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

Issue 2832873002: Fix two analyzer warnings. (Closed)

Created:
3 years, 8 months ago by devoncarew
Modified:
3 years, 8 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : revert statement_completion.dart change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M pkg/analyzer/test/generated/gn_test.dart View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 8 (2 generated)
devoncarew
3 years, 8 months ago (2017-04-20 17:17:33 UTC) #2
messick
https://codereview.chromium.org/2832873002/diff/1/pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart File pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart (left): https://codereview.chromium.org/2832873002/diff/1/pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart#oldcode241 pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart:241: bool _complete_controlFlowBlock() { You could fix the warning by ...
3 years, 8 months ago (2017-04-20 17:30:50 UTC) #3
devoncarew
https://codereview.chromium.org/2832873002/diff/1/pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart File pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart (left): https://codereview.chromium.org/2832873002/diff/1/pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart#oldcode241 pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart:241: bool _complete_controlFlowBlock() { On 2017/04/20 17:30:49, messick wrote: > ...
3 years, 8 months ago (2017-04-20 17:37:36 UTC) #4
messick
On 2017/04/20 17:37:36, devoncarew wrote: > https://codereview.chromium.org/2832873002/diff/1/pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart > File > pkg/analysis_server/lib/src/services/completion/statement/statement_completion.dart > (left): > > ...
3 years, 8 months ago (2017-04-20 17:42:26 UTC) #5
Brian Wilkerson
lgtm
3 years, 8 months ago (2017-04-20 18:30:06 UTC) #6
devoncarew
3 years, 8 months ago (2017-04-20 18:32:55 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
3998e356790de28809c4d260606a5b4d4b1bbadb (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698