Skip to content

Conversation

@mit-mit
Copy link
Member

@mit-mit mit-mit commented Jan 9, 2019

Wrap lines consistently with other files

@mit-mit mit-mit requested a review from Hixie January 9, 2019 12:39
@zoechi zoechi added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Jan 9, 2019
@gspencergoog
Copy link
Contributor

LGTM, but will this affect the license compression that we use in the engine? It relies on licenses having common segments of text to make the binary size smaller. Check with @goderbauer.

32384589-a60f0e74-c078-11e7-9bc1-e5b5287aea9d

mit-mit added a commit to flutter/engine that referenced this pull request Jan 10, 2019
@mit-mit
Copy link
Member Author

mit-mit commented Jan 10, 2019

Sent a PR to synchronize flutter/engine

@mit-mit mit-mit requested a review from goderbauer January 10, 2019 10:44
@goderbauer
Copy link
Member

Can you build a flutter apk and check that this change doesn't increase the size of the combined LICENSE file inside?

@mit-mit
Copy link
Member Author

mit-mit commented Jan 29, 2019

Change in flutter/engine landed. I don't see any size difference when flutter build apk'ing the gallery

@mit-mit mit-mit merged commit 9499cb1 into master Jan 29, 2019
@mit-mit mit-mit deleted the mit-mit-patch-1 branch January 29, 2019 15:13
kangwang1988 pushed a commit to XianyuTech/flutter that referenced this pull request Feb 12, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants