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

Issue 2978903002: Remove number from operator mismatch message. (Closed)

Created:
3 years, 5 months ago by ahe
Modified:
3 years, 5 months ago
Reviewers:
Johnni Winther
CC:
reviews_dartlang.org, dart-uxr+reviews_google.com, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : Handle minus operators. #

Patch Set 3 : Add missing show clause. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+124 lines, -28 lines) Patch
M pkg/front_end/lib/src/fasta/fasta_codes_generated.dart View 1 2 1 chunk +83 lines, -14 lines 0 comments Download
M pkg/front_end/lib/src/fasta/source/outline_builder.dart View 1 2 2 chunks +27 lines, -5 lines 0 comments Download
M pkg/front_end/messages.yaml View 1 2 2 chunks +14 lines, -4 lines 0 comments Download
M pkg/front_end/tool/_fasta/generate_messages.dart View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
ahe
3 years, 5 months ago (2017-07-13 10:48:48 UTC) #2
Johnni Winther
lgtm https://codereview.chromium.org/2978903002/diff/1/pkg/front_end/lib/src/fasta/source/outline_builder.dart File pkg/front_end/lib/src/fasta/source/outline_builder.dart (right): https://codereview.chromium.org/2978903002/diff/1/pkg/front_end/lib/src/fasta/source/outline_builder.dart#newcode398 pkg/front_end/lib/src/fasta/source/outline_builder.dart:398: template = templateOperatorParameterMismatch2; How do you handle `operator ...
3 years, 5 months ago (2017-07-13 11:23:39 UTC) #3
ahe
Thank you, Johnni. I'll get back on this. https://codereview.chromium.org/2978903002/diff/1/pkg/front_end/lib/src/fasta/source/outline_builder.dart File pkg/front_end/lib/src/fasta/source/outline_builder.dart (right): https://codereview.chromium.org/2978903002/diff/1/pkg/front_end/lib/src/fasta/source/outline_builder.dart#newcode398 pkg/front_end/lib/src/fasta/source/outline_builder.dart:398: template ...
3 years, 5 months ago (2017-07-13 11:32:19 UTC) #4
ahe
PTAL
3 years, 5 months ago (2017-07-17 11:44:16 UTC) #6
Johnni Winther
lgtm
3 years, 5 months ago (2017-07-17 12:50:57 UTC) #7
ahe
3 years, 5 months ago (2017-07-18 12:43:20 UTC) #9
Message was sent while issue was closed.
Committed patchset #3 (id:60001) manually as
456571978307e43b22a7be46a9d81519e012c0c8 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698