Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't mark whole class #362

Closed
floitschG opened this issue Nov 7, 2011 · 4 comments
Closed

Don't mark whole class #362

floitschG opened this issue Nov 7, 2011 · 4 comments
Assignees
Labels
closed-duplicate Closed in favor of an existing report

Comments

@floitschG
Copy link
Contributor

When a class doesn't fully implement an interface then the whole class body (including the method bodies) is marked (with yellow underlines).
It would be much nicer if only the header would be marked.

@bwilkerson
Copy link
Member

This is actually a problem in the compiler. The CompilationError that is being produced in this case should have it's source range set to the source range for the name of the class, not the whole file.


Removed Area-Editor label.
Added Area-Compiler label.

@scheglov
Copy link
Contributor

Can you provide test case?

This code compiles without any warning at all.

                interface Foo {
                  void foo();
                }
                class Test implements Foo {
                }

@floitschG
Copy link
Contributor Author

There seems to be another bug in DartC. Apparently you need to add a constructor to get the warning. Also you need to instantiate the class.

interface Foo {
  void foo();
}
class Test implements Foo {
  Test();
}
bar() {
  return new Test();
}

@scheglov
Copy link
Contributor

This will be (or not) fixed as
http://code.google.com/p/dart/issues/detail?id=375


Set owner to @scheglov.
Added Duplicate label.
Marked as being merged into #375.

@floitschG floitschG added Type-Defect closed-duplicate Closed in favor of an existing report labels Nov 28, 2011
copybara-service bot pushed a commit that referenced this issue May 18, 2022
Changes:
```
> git log --format="%C(auto) %h %s" 9669926..9145f30
 https://dart.googlesource.com/intl.git/+/9145f30 Bump actions/checkout from 2 to 3 (#462)
 https://dart.googlesource.com/intl.git/+/f545753 update the analysis_options.yaml file (#449)
 https://dart.googlesource.com/intl.git/+/fee2c2d Add a changelog entry for a recent feature (#406)
 https://dart.googlesource.com/intl.git/+/4e37662 GitHub Sync (#405)
 https://dart.googlesource.com/intl.git/+/0a14483 Bump dart-lang/setup-dart from 0.3 to 1 (#403)
 https://dart.googlesource.com/intl.git/+/f22f7b5 fix directive sorting
 https://dart.googlesource.com/intl.git/+/ed2fbe0 Add dependabot
 https://dart.googlesource.com/intl.git/+/2518923 Merge pull request #377 from dart-lang/franklinyow-patch-1
 https://dart.googlesource.com/intl.git/+/bee0456 Update LICENSE
 https://dart.googlesource.com/intl.git/+/9bb4bcc Migrating to dart:ffi in Dart 2.12.
 https://dart.googlesource.com/intl.git/+/532a1e3 Migrating to `dart:ffi` in Dart 2.12 and `package:ffi` 1.0.0.
 https://dart.googlesource.com/intl.git/+/458129d Replace git dependencies with published versions
 https://dart.googlesource.com/intl.git/+/5fd7a11 COPYBARA CHANGE FOR dart-archive/intl#358
 https://dart.googlesource.com/intl.git/+/2c8e014 Fix typo; see https://screenshot.googleplex.com/5afwL4iQtHeJKLC
 https://dart.googlesource.com/intl.git/+/33251e0 Prepare to publish for stable null safety (#362)
 https://dart.googlesource.com/intl.git/+/b3da438 Prep release for beta (#338)
 https://dart.googlesource.com/intl.git/+/48675cf Prepare a version of intl that supports the 2.12 sdk

```

Diff: https://dart.googlesource.com/intl.git/+/9669926609e7efc17dfd74fbb44ec719a7e573cc~..9145f308f1458f37630a1ffce3b7d3b471ebbc56/
Change-Id: Ic431da0e0785bd4ece86542fae30aad55f384f50
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/245222
Commit-Queue: Devon Carew <devoncarew@google.com>
Reviewed-by: Nate Bosch <nbosch@google.com>
copybara-service bot pushed a commit that referenced this issue May 30, 2023
…buf, shelf, test, tools, vector_math, webdev

Revisions updated by `dart tools/rev_sdk_deps.dart`.

dartdoc (https://github.com/dart-lang/dartdoc/compare/59dc475..1d94484):
  1d94484c  2023-05-29  dependabot[bot]  Bump github/codeql-action from 2.3.3 to 2.3.5 (#3422)
  0edc1a71  2023-05-28  dependabot[bot]  Bump http from 0.13.6 to 1.0.0 (#3421)

http (https://github.com/dart-lang/http/compare/dfec389..8a4a4a6):
  8a4a4a6  2023-05-25  Brian Quinlan  Add a better toString to _ClientSocketException (#948)
  5c1f1ad  2023-05-25  Devon Carew  regenerate with the latest mono_repo (#947)

leak_tracker (https://github.com/dart-lang/leak_tracker/compare/9c6e9b3..7f2cab3):
  7f2cab3  2023-05-26  Polina Cherkasova  Nicely format retaining path. (#68)

lints (https://github.com/dart-lang/lints/compare/72f107a..4236c43):
  4236c43  2023-05-26  Parker Lougheed  Remove pedantic from README (#124)
  4ac79d8  2023-05-24  Parker Lougheed  Update example for latest lints version (#123)

mockito (https://github.com/dart-lang/mockito/compare/153c145..40fe2ca):
  40fe2ca  2023-05-25  Nate Bosch  Expand constraint on package:http

native (https://github.com/dart-lang/native/compare/45e16dc..76bc55e):
  76bc55e  2023-05-30  Daco Harkes  [c_compiler] Target ios_x64 (#53)

protobuf (https://github.com/dart-lang/protobuf/compare/7d2d293..346a72d):
  346a72d  2023-05-30  Ömer Sinan Ağacan  Fix generated ignore_for_file directives (#833)
  35ea45f  2023-05-26  Kevin Moore  Latest mono_repo (#834)

shelf (https://github.com/dart-lang/shelf/compare/56919a1..a404b6a):
  a404b6a  2023-05-25  Devon Carew  re-generate w/ the latest monorepo (#362)

test (https://github.com/dart-lang/test/compare/309596e..3276921):
  32769215  2023-05-25  dependabot[bot]  Bump github/codeql-action from 2.3.2 to 2.3.5 (#2023)
  f74e85c8  2023-05-25  dependabot[bot]  Bump dart-lang/setup-dart from 1.3.0 to 1.5.0 (#2022)
  4b2bd272  2023-05-25  Devon Carew  update the mono_repo and dependabot configs (#2021)

tools (https://github.com/dart-lang/tools/compare/81ff996..b90a7e8):
  b90a7e8  2023-05-26  Devon Carew  blast_repo fixes (#106)

vector_math (https://github.com/google/vector_math.dart/compare/e3de8da..cd87f57):
  cd87f57  2023-05-30  JKris95  Axis calculation of quaternions from small angles (#272)
  3762b25  2023-05-30  Lukas Klingsbo  Removes the `new` keyword from readme (#284)
  df5877f  2023-05-30  Lukas Klingsbo  Use named constructors in Vector2 and some general optimizations (#289)

webdev (https://github.com/dart-lang/webdev/compare/d74fadd..4b69f1d):
  4b69f1dd  2023-05-26  Anna Gringauze  fix format breaking tests (#2124)
  b75f8e62  2023-05-25  Devon Carew  re-generate w/ the latest monorepo (#2121)

Change-Id: Ide9b7781102b654db15114d01cd4fbca40478906
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/306304
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-duplicate Closed in favor of an existing report
Projects
None yet
Development

No branches or pull requests

3 participants