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

Documentation for the dart:jni library #2774

Merged
merged 1 commit into from
Jun 30, 2016
Merged

Documentation for the dart:jni library #2774

merged 1 commit into from
Jun 30, 2016

Conversation

jason-simmons
Copy link
Member

No description provided.

Hixie
Hixie reviewed Jun 28, 2016
View reviewed changes
flutter/lib/jni/jni.dart
/// as arguments to wrapped Java methods. The JNI libraries will convert these arguments
/// from Dart types to the matching Java type.
///
/// Note that Dart does not support function overloading, and as a result the Java method
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: strike "note that"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@Hixie
Copy link
Contributor

Hixie commented Jun 28, 2016

LGTM with the additional fleshing out suggested in my comments.

@abarth may wish to review this for accuracy, I only reviewed for style.

@jason-simmons jason-simmons merged commit 2eb81e3 into flutter:master Jun 30, 2016
@jason-simmons jason-simmons mentioned this pull request Jun 30, 2016
Closed
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
@jonasfj
Roll pub
41381e9 
a7e448b pub upgrade --major-versions (flutter#2784)
06bcff0 pub upgrade foo, only unlocks foo from pubspec.lock (flutter#2781)
aa20a4b Usage exceptions no stacktrace (flutter#2776)
14e12bf Clarify error message when spawning a subprocess fails (flutter#2763)
5ef0a0e Added a pub upgrade --nullsafety mode (flutter#2741)
9cde240 Fix pub remove --dry-run (flutter#2774)
7426be9 Remove hack now issue has been fixed (flutter#2773)
90a1f77 Limit retries in test to avoid timeout (flutter#2765)
3aa327c Stop depending on .packages (flutter#2764)

Change-Id: Ifd8a8402c4628e7278b6fedfd2ca7902004fa6e8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/175241
Auto-Submit: Jonas Jensen <jonasfj@google.com>
Reviewed-by: Sigurd Meldgaard <sigurdm@google.com>
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
@davidmorgan
Revert "Roll pub"
c0e13c1 
This reverts commit 41381e9.

Reason for revert: pub fails on startup if git is not installed; b/175080640

Original change's description:
> Roll pub
>
> a7e448b pub upgrade --major-versions (flutter#2784)
> 06bcff0 pub upgrade foo, only unlocks foo from pubspec.lock (flutter#2781)
> aa20a4b Usage exceptions no stacktrace (flutter#2776)
> 14e12bf Clarify error message when spawning a subprocess fails (flutter#2763)
> 5ef0a0e Added a pub upgrade --nullsafety mode (flutter#2741)
> 9cde240 Fix pub remove --dry-run (flutter#2774)
> 7426be9 Remove hack now issue has been fixed (flutter#2773)
> 90a1f77 Limit retries in test to avoid timeout (flutter#2765)
> 3aa327c Stop depending on .packages (flutter#2764)
>
> Change-Id: Ifd8a8402c4628e7278b6fedfd2ca7902004fa6e8
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/175241
> Auto-Submit: Jonas Jensen <jonasfj@google.com>
> Reviewed-by: Sigurd Meldgaard <sigurdm@google.com>
> Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>

TBR=sigurdm@google.com,jonasfj@google.com

Change-Id: I29b20b4a2dc037146dd27c8fe6e6ecce6833e7b6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/175303
Reviewed-by: David Morgan <davidmorgan@google.com>
Commit-Queue: David Morgan <davidmorgan@google.com>
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
@sigurdm
Reland "Roll pub"
bf26eb3 
This is a reland of 41381e9

Contains one more fix:
8ea9612 fix tryGitCommand

Original change's description:
> Roll pub
>
> a7e448b pub upgrade --major-versions (flutter#2784)
> 06bcff0 pub upgrade foo, only unlocks foo from pubspec.lock (flutter#2781)
> aa20a4b Usage exceptions no stacktrace (flutter#2776)
> 14e12bf Clarify error message when spawning a subprocess fails (flutter#2763)
> 5ef0a0e Added a pub upgrade --nullsafety mode (flutter#2741)
> 9cde240 Fix pub remove --dry-run (flutter#2774)
> 7426be9 Remove hack now issue has been fixed (flutter#2773)
> 90a1f77 Limit retries in test to avoid timeout (flutter#2765)
> 3aa327c Stop depending on .packages (flutter#2764)
>
> Change-Id: Ifd8a8402c4628e7278b6fedfd2ca7902004fa6e8
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/175241
> Auto-Submit: Jonas Jensen <jonasfj@google.com>
> Reviewed-by: Sigurd Meldgaard <sigurdm@google.com>
> Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>

Change-Id: Id50e6eca0372860f7087882a5c67141def07031c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/175309
Reviewed-by: Jonas Jensen <jonasfj@google.com>
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
@jonasfj
Fix pub remove --dry-run (flutter#2774)
9cde240 
* Fix case where `pub remove foo --dry-run` might update `PUB_CACHE/global_packages/`

* Update add.dart
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jason-simmons @Hixie