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

Cant Spawn Isolates from static method #3011

Closed
DartBot opened this issue May 11, 2012 · 9 comments
Closed

Cant Spawn Isolates from static method #3011

DartBot opened this issue May 11, 2012 · 9 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-obsolete Closed as the reported issue is no longer relevant library-isolate

Comments

@DartBot
Copy link

DartBot commented May 11, 2012

This issue was originally filed by @pjako


What steps will reproduce the problem?

import("dart:isolate");

class Logic {
  static void isolate() {
    spawnFunction(spawn);
  }
  static void spawn() {
  }

}
void main() {
  Logic.isolate();
}
What is the expected output? What do you see instead?

Unhandled exception:
Illegal argument(s): spawnFunction expects to be passed a closure to a top-level static function
 0. Function: '::_spawnFunction@6be832b' url: 'dart:isolate' line:383 col:1
 1. Function: '::spawnFunction' url: 'dart:isolate' line:36 col:24
 2. Function: 'Logic.isolate' url: 'file:///Users/work4/dart/Logic/Logic.dart' line:4 col:18
 3. Function: '::main' url: 'file:///Users/work4/dart/Logic/Logic.dart' line:11 col:16

What version of the product are you using? On what operating system?
Dart SDK 7405
Mac OSX

@sethladd
Copy link
Contributor

The docs read: "The topLevelFunction argument must be a static top-level function or a static method that takes no arguments."

http://api.dartlang.org/dart_isolate.html


cc @sigmundch.
Added Area-Library, Isolates, Triaged labels.

@sigmundch
Copy link
Member

Thanks for the error report. This is a bug we have in all implementations so far (vm and frog).


cc @turnidge.

@sigmundch
Copy link
Member

Added Library-Isolates label.

@sigmundch
Copy link
Member

Removed Isolates label.

@DartBot
Copy link
Author

DartBot commented Sep 26, 2012

This comment was originally written by [email protected]


Friendly ping to check not slipped between cracks. Does absence of M1 tag mean "don't expect it soon" ?

Also, am I correct in thinking that without this, one can only spawnFunction() with a top-level function?

Thanks.

@dgrove
Copy link
Contributor

dgrove commented Jan 11, 2013

Added Library-Isolate label.

@dgrove
Copy link
Contributor

dgrove commented Jan 11, 2013

Removed Library-Isolates label.

@lrhn
Copy link
Member

lrhn commented Nov 12, 2013

Issue #7698 has been merged into this issue.

@lrhn
Copy link
Member

lrhn commented Aug 4, 2014

This seems to work in both the VM and dart2js now.
This code runs and prints "go":


import "dart:isolate";
class C { static void run(string) { print(string); } }
main() { Isolate.spawn(C.run, "go"); }



Added AssumedStale label.

@DartBot DartBot added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-isolate closed-obsolete Closed as the reported issue is no longer relevant labels Aug 4, 2014
copybara-service bot pushed a commit that referenced this issue May 20, 2022
…25 revisions)

https://dart.googlesource.com/dartdoc/+log/334072b0cad4..cf0685a2d1ce

2022-05-19 [email protected] Create scorecards-analysis.yml (#3032)
2022-05-18 [email protected] Add the docs.dart.js.map file to published package (#3042)
2022-05-17 [email protected] Reintroduce query param search (#3026)
2022-05-17 [email protected] Include docs.dart.js in published code (#3040)
2022-05-17 [email protected] Optimize built js (#3028)
2022-05-17 [email protected] Use dart run instead of deprecated dart pub run (#3038)
2022-05-17 [email protected] fix exception for empty dartdoc_options.yaml file (#3034)
2022-05-16 [email protected] Fix build config (#3035)
2022-05-02 [email protected] Drop version back to 5.1.0 (#3031)
2022-05-02 [email protected] Fixed: YouTube directive respects width/height argument (#3030)
2022-04-19 [email protected] Update pubspec.yaml (#3025)
2022-04-12 [email protected] Remove dead field commentRefs, which is always empty (#3021)
2022-04-12 [email protected] make the converted web frontend follow Dart conventions (#3020)
2022-04-11 [email protected] convert lib/resources/script.js to Dart (#3016)
2022-04-11 49699333+dependabot[bot]@users.noreply.github.com Bump actions/cache from 3.0.1 to 3.0.2 (#3019)
2022-04-10 [email protected] rename the CI script (#3017)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump actions/cache from 2 to 3.0.1 (#3014)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump markdown from 4.0.1 to 5.0.0 (#3015)
2022-04-08 [email protected] Move many static declarations to top-level; avoid RegExp (#3009)
2022-04-08 49699333+dependabot[bot]@users.noreply.github.com Bump actions/checkout from 2 to 3 (#3013)
2022-04-07 [email protected] Add a web frontend (#3011)
2022-04-07 [email protected] enable dependabot deps checks (#3012)
2022-04-07 [email protected] Do not store the AstNode on ModelNode; eagerly grab the source indexes (#3008)
2022-04-07 [email protected] Create a .pubignore in order to skip testing files from publish check (#3010)
2022-03-29 [email protected] Bump to version 5.1.0 (#3006)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-doc-dart-sdk
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Dart Documentation Generator: https://github.com/dart-lang/dartdoc/issues
To file a bug in Dart SDK: https://github.com/dart-lang/sdk/issues

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Tbr: [email protected]
Change-Id: I3467a27d6947c43f7f41acbb2959d72863fd8a34
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/245483
Commit-Queue: Devon Carew <[email protected]>
Reviewed-by: 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
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-obsolete Closed as the reported issue is no longer relevant library-isolate
Projects
None yet
Development

No branches or pull requests

5 participants