[guice] Merge internal changes (#919)
2 views
Skip to first unread message
Sam Berlin
Apr 23, 2015, 3:27:18 PM4/23/15
to google/guice
You can view, comment on, or merge this pull request online at:
https://github.com/google/guice/pull/919
Commit Summary
- Also fix usages of a couple of @Beta APIs.
- Remove the snapshot repository that was configured to support dagger-adapter. Dagger 2 is now released, so this is not necessary.
File Changes
- M core/src/com/google/inject/internal/CycleDetectingLock.java (18)
- M extensions/service/src/com/google/inject/service/AsyncService.java (9)
- M pom.xml (13)
Patch Links:
—
Reply to this email directly or view it on GitHub.
Colin Decker
Apr 23, 2015, 3:55:08 PM4/23/15
to google/guice
LGTM other than the indentation issue, which can be fixed separately.
Sam Berlin
Apr 23, 2015, 4:08:05 PM4/23/15
to google/guice
Merged #919.
Reply all
Reply to author
Forward
0 new messages