-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
scriptTag should be at offset zero #2259
Labels
area-language
Dart language related items (some items might be better tracked at github.com/dart-lang/language).
Comments
peter-ahe-google
added
Type-Defect
area-language
Dart language related items (some items might be better tracked at github.com/dart-lang/language).
labels
Apr 12, 2012
copybara-service bot
pushed a commit
that referenced
this issue
Oct 24, 2023
Revisions updated by `dart tools/rev_sdk_deps.dart`. dartdoc (https://github.com/dart-lang/dartdoc/compare/f7e9b17..53da3e1): 53da3e1d 2023-10-23 dependabot[bot] Bump actions/checkout from 4.1.0 to 4.1.1 (#3544) 62d5469d 2023-10-23 Sam Rawlins Support extension types in sidebars and categories (#3537) 69abd0f0 2023-10-22 Sam Rawlins Refactor file-gathering logic (#3539) e9c61d1d 2023-10-22 Sam Rawlins Convert some late final fields to getters in Accessor, Annotation, Constructor (#3534) adcdc8b7 2023-10-20 Parker Lougheed Update package:lints to v3 (#3542) 41f02622 2023-10-19 Sam Rawlins Greatly simplify ModelCommentReference (#3541) 50e4b679 2023-10-19 Sam Rawlins Remove use of NodeLocator2 (#3538) b2de813c 2023-10-19 Sam Rawlins Remove support for deprecated leading new in comment references (#3529) b350c688 2023-10-19 Sam Rawlins Bump to 7.0.1 (#3540) 5256e2fb 2023-10-19 Sam Rawlins Remove unused warnings (#3533) lints (https://github.com/dart-lang/lints/compare/975c687..2cf8403): 2cf8403 2023-10-23 Devon Carew add no_wildcard_variable_uses; rev to a new major version (#165) 7b0f556 2023-10-23 Parker Lougheed Remove mention of no_wildcard_variable_uses from changelog (#164) matcher (https://github.com/dart-lang/matcher/compare/356e5f6..7512f80): 7512f80 2023-10-23 Michael Goderbauer Specify language in neverCalled docs for nicer formatting in docs (#230) protobuf (https://github.com/dart-lang/protobuf/compare/050c162..3528fad): 3528fad 2023-10-24 Ömer Sinan Ağacan Use `setRange` when copying output chunks to the final buffer in `CodedBufferWriter` (#887) tools (https://github.com/dart-lang/tools/compare/15cc9c7..da6bb18): da6bb18 2023-10-24 Elias Yishak Enum + event constructors added for doctor events (#178) e3dd149 2023-10-24 Elias Yishak Use futures list internally to manage send events (#184) webdev (https://github.com/dart-lang/webdev/compare/1bd434b..6e324af): 6e324afb 2023-10-24 Ben Konyi Add dependency on `package:vm_service_interface` (#2262) 8429a79f 2023-10-20 Elliott Brooks Only notify chatroom when daily stable testing fails (#2259) 3463d169 2023-10-19 Elliott Brooks Remove Chrome 115 extension error (#2258) fdebc06e 2023-10-19 Elliott Brooks Reset Webdev to 3.2.0-wip after release (#2260) 9cffb896 2023-10-19 Elliott Brooks Reset DWDS to 22.1.0-wip after release (#2256) 07c70c6c 2023-10-19 Elliott Brooks Prepare Webdev for release to 3.1.0 (#2255) Change-Id: Ieae3aadcc804a270867d7935b702987cf1f6d51c Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/332060 Reviewed-by: Elias Yishak <[email protected]> Commit-Queue: 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-language
Dart language related items (some items might be better tracked at github.com/dart-lang/language).
The specification does not mention that a scriptTag must be at offset zero in the compilation unit.
However, scriptTags are useless if not the first two bytes. A Unix kernel does only look at the first two bytes.
Since #! is a fairly arcane feature, I think it is generally in our users interest that we help them avoid silly mistakes such as adding a copyright before #!.
The text was updated successfully, but these errors were encountered: