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.

Sign up for GitHub

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

Jump to bottom

frog not reporting missing abstract methods #370

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

frog not reporting missing abstract methods #370

DartBot opened this issue Nov 7, 2011 · 4 comments
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue web-dart2js

Comments

@DartBot
Copy link

DartBot commented Nov 7, 2011

This issue was originally filed by [email protected]

Frog does not report if an abstract method is not implemented.
@stevemessick
Copy link
Contributor

Added Area-Frog, Triaged labels.

@jmesserly
Copy link

FYI, I posted about this here:
http://groups.google.com/a/dartlang.org/group/compiler-dev/browse_thread/thread/c9d9ec29b0974d4f#

I'm not 100% sure we want to make this a warning, given the false-positives it causes in our code. I do have a change that does it here:
http://codereview.chromium.org/8618006

@anders-sandholm
Copy link
Contributor

Removed Area-Frog label.
Added Area-Dart2JS, FromAreaFrog labels.

@kasperl
Copy link

kasperl commented Jun 12, 2012

Added WontFix label.

@kevmoo kevmoo added closed-not-planned Closed as we don't intend to take action on the reported issue and removed resolution-wont_fix labels Mar 1, 2016
dart-bot pushed a commit that referenced this issue Jun 12, 2019
@athomas
[co19] Roll co19 to cd1be9d07df32413024fe5c4ae9c5c6017c2f2c8
299532d 
2019-06-11 [email protected] More fixes to cast response to Stream<List<int>> (#385)
2019-06-11 [email protected] Pass Uint8List to Datagram (#382)
2019-06-11 [email protected] Cast HttpRequest and HttpClientResponse streams to List<int> (#384)
2019-06-10 [email protected] Fixes #364. Change expected result to null for default values of clientMaxWindowBits and serverMaxWindowBits
2019-06-07 [email protected] Fixes #380. Added tests for logical and bitwise operations via type aliases
2019-06-07 [email protected] Additional fix for #373. Code style improved and more strict pattern checking
2019-06-06 [email protected] Fix for #380. Added tests for type aliases for built-in types. Test string concatenation and arithmetic operations
2019-06-05 [email protected] Fixes #380. Added tests for type aliases for built-in types initialization
2019-06-05 [email protected] Fixes #373. Expect reasonable file mode on Unix
2019-06-04 [email protected] Fixes #379. Numerous fixes for io/Process tests
2019-06-04 [email protected] Fixes #378. Use Platform.resolvedExecutable instead of 'dart' command
2019-06-04 [email protected] Fixed Issue #375: it's possible that IPv6 loopback does not exist.
2019-06-04 [email protected] Fixed Issue #236: host.host can be either "localhost" or Platform.localHostname.
2019-06-04 [email protected] Issue #370, instantiate-to-bounds: added static tests for non-function type aliases.
2019-06-04 [email protected] Fix for #377. Use correct network interface type name (IPvX)
2019-06-04 [email protected] Fixes #374. Change pattern for error messages
2019-06-04 [email protected] Issue #370, instantiate-to-bounds: added dynamic tests for non-function type aliases.
2019-06-04 [email protected] Fix for #372. Expect.fail() on timeout added
2019-06-04 [email protected] Fixes #371. Remove excessive asyncStart()
2019-06-03 [email protected] Issue #147, test super bounded types: added tests for non-function type aliases.
2019-05-30 [email protected] Fixes #369. Use correct type arguments to avoid errors
2019-05-30 [email protected] Fixes #368. Don't try to bind system port
2019-05-30 [email protected] Fixes #367. Remove excessive asyncStart()
2019-05-30 [email protected] Fixes #366. Change expected result to SocketException

[email protected]

Change-Id: I137689755907b3333e597d7d210db2b4d37d70d0
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/105641
Reviewed-by: Alexander Thomas <[email protected]>
Reviewed-by: William Hesse <[email protected]>
This was referenced Oct 29, 2020
Open
Closed
Closed
This was referenced Nov 5, 2020
Closed
Open
copybara-service bot pushed a commit that referenced this issue Aug 7, 2023
@devoncarew
[deps] rev ecosystem, http, mockito, native, protobuf, shelf, sse, te…
1b1c62a 
…st, tools

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

ecosystem (https://github.com/dart-lang/ecosystem/compare/97fc1a7..dfeda1a):
  dfeda1a  2023-08-01  dependabot[bot]  Bump actions/labeler from 4.0.4 to 4.3.0 (#148)
  c341051  2023-08-01  dependabot[bot]  Bump coverallsapp/github-action from 2.2.0 to 2.2.1 (#147)
  c1c8d1f  2023-08-01  Moritz  Update health.yaml (#146)
  31c5d21  2023-07-27  Devon Carew  misc updates to the label management tool (#145)

http (https://github.com/dart-lang/http/compare/4289e8b..7e9ed12):
  7e9ed12  2023-08-01  dependabot[bot]  Bump actions/labeler from 4.2.0 to 4.3.0 (#1000)
  e8e35db  2023-07-31  Alex James  [java_http] send request body (#995)

mockito (https://github.com/dart-lang/mockito/compare/b421775..ff79de6):
  ff79de6  2023-08-04  Nate Bosch  Allow the latest package:analyzer

native (https://github.com/dart-lang/native/compare/f0dc3e9..0187d0e):
  0187d0e  2023-08-07  Daco Harkes  [native_assets_cli] Rename `Asset` `name` to `id` (#113)
  1b984c7  2023-08-07  Daco Harkes  [native_assets_cli] Replace `TypeError`s with `FormatException`s (#112)
  b2b26db  2023-08-07  Daco Harkes  [native_assets_builder] Stop throwing from BuildRunner (#108)
  c940ac8  2023-08-03  Daco Harkes  [native_assets_builder] return build dependencies (#107)
  63daab8  2023-08-03  Daco Harkes  [native_assets_builder] Fix dry run directory structure (#110)
  3f26f20  2023-08-01  dependabot[bot]  Bump coverallsapp/github-action from 2.2.0 to 2.2.1 (#104)

protobuf (https://github.com/dart-lang/protobuf/compare/d9e8a31..217c030):
  217c030  2023-08-02  Ömer Sinan Ağacan  Avoid holding onto the buffer when parsing unknown length-delimited fields (#863)

shelf (https://github.com/dart-lang/shelf/compare/bd59ead..73edd2b):
  73edd2b  2023-08-01  dependabot[bot]  Bump actions/labeler from 4.2.0 to 4.3.0 (#370)

sse (https://github.com/dart-lang/sse/compare/e241085..8cc5b11):
  8cc5b11  2023-08-01  dependabot[bot]  Bump nanasess/setup-chromedriver from 2.0.0 to 2.1.1 (#86)

test (https://github.com/dart-lang/test/compare/92eb0f7..5d571d6):
  5d571d64  2023-08-03  Nate Bosch  Add --fail-fast flag (#2040)
  a9dcce29  2023-08-01  Nate Bosch  Rerun publish workflow on more PR changes (#2070)
  50d558b2  2023-08-01  dependabot[bot]  Bump github/codeql-action from 2.20.1 to 2.21.2 (#2071)
  9e124e9f  2023-07-31  Nate Bosch  Prepare to publish (#2069)

tools (https://github.com/dart-lang/tools/compare/af3fc99..f14bf2e):
  f14bf2e  2023-08-07  Elias Yishak  Remove unused NoOp classes (#138)
  a2aa1c3  2023-08-07  Devon Carew  add CI; update readme (#140)
  d424568  2023-08-06  Devon Carew  add usage docs to package:cli_config (#141)
  921611a  2023-08-03  Elias Yishak  Survey handler feature (#109)

Change-Id: Iee720272733822d11caf46adcae516f12f11abec
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/318801
Commit-Queue: Devon Carew <[email protected]>
Reviewed-by: Konstantin Shcheglov <[email protected]>
Auto-Submit: Devon Carew <[email protected]>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@kevmoo @jmesserly @kasperl @anders-sandholm @stevemessick @DartBot