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

Moe sync 11/15/2016 #498

Merged
merged 3 commits into from
Nov 16, 2016
Merged

Moe sync 11/15/2016 #498

merged 3 commits into from
Nov 16, 2016

Conversation

ronshapiro
Copy link

No description provided.

@gk5885
Copy link

gk5885 commented Nov 15, 2016

LGTM!

ronshapiro and others added 3 commits November 16, 2016 16:08
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=139125765
…ublicly accessible way to invoke package-private methods and constructors. Then, use those proxy methods directly to fulfill requests that don't require Providers.

The interaction of this behavior + the existing behavior to invoke accessible elements directly yields 861 proxy accesses and 511 direct accesses in []' CommonDataReloadComponent for example.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=139167112
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=139355153
@ronshapiro ronshapiro merged commit b4811d0 into master Nov 16, 2016
@ronshapiro ronshapiro deleted the moe_sync_11/15/2016 branch November 16, 2016 22:21
@JakeWharton
Copy link

Interesting stuff!

On Wed, Nov 16, 2016 at 5:21 PM Ron Shapiro [email protected]
wrote:

Merged #498 #498.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#498 (comment), or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAEEEWcX5c7kSutoZxS5bCYpeWKQ35oeks5q-4HOgaJpZM4KzCa6
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants