This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Implement 'stretch' for flexible items.#698
Merged
eseidelGoogle merged 1 commit intoflutter:masterfrom Aug 19, 2015
Merged
Conversation
Contributor
|
lgtm |
eseidelGoogle
added a commit
that referenced
this pull request
Aug 19, 2015
Implement 'stretch' for flexible items.
Hixie
added a commit
to Hixie/sky_engine
that referenced
this pull request
Aug 21, 2015
Fixes flutter#698 to actually work. Also, adds some debugging aids around Flex. And a test to check this fix.
rhencke
pushed a commit
to rhencke/engine
that referenced
this pull request
Dec 20, 2020
…ter#698) * Attempt to uniformly format code with or without "new"/"const". One problem with optional new is that dartfmt can no longer reliably tell what's a constructor call versus some other kind of invocation (usually a static method on a class). In particular, named constructor calls do not get formatted as part of a method chain. You get this: new Foo().named() .method() .method() .method(); Not: new Foo() .named() .method() .method() .method(); But if that were a static method call, you *would* get: Foo() .named() .method() .method() .method(); This means that removing the "new"/"const" keywords from your code can change how its formatted (beyond just the changes caused by the lines being shorter). In particular, it means if you run: dartfmt --fix -w . dartfmt -w . The second invocation may produce changes, even though it should be strictly redundant. That's because the first call *does* know which calls are constructors because the keywords are still there when it's parsed, but the second does not. That violates an expectation that dartfmt is idempotent. This addresses that by having a uniform set of formatting rules for constructors and other invocations. To avoid ugly cases where a named constructor would get slurped into a method chain, it pulls out all static calls from method chains. So with this change you *would* get: Foo().named() .method() .method() .method(); It does this for all static calls, not just constructors. That makes this a fairly large change. I've run it on a large corpus, and I actually think it looks pretty good with that style. There is no fully reliably way to identify a "static" call just from syntax. Instead, it uses the heuristic that class names are capitalized (but not all caps). This seems to work correctly on all but the weirdest code I've seen in the wild. * Update CHANGELOG.
rhencke
pushed a commit
to rhencke/engine
that referenced
this pull request
Dec 20, 2020
Random VK_ANDROID_external_memory_android_hardware_buffer cleanup
rhencke
pushed a commit
to rhencke/engine
that referenced
this pull request
Dec 20, 2020
Reapply flutter#698: VK_ANDROID_external_memory_android_hardware_buffer markup cleanup
itekdev
pushed a commit
to itekdev/flutter-engine0
that referenced
this pull request
May 1, 2023
Bump actions/checkout from 3.4.0 to 3.5.0
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.