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

Implement Canvas.drawPoints #2748

Merged
merged 2 commits into from
Jun 13, 2016
Merged

Implement Canvas.drawPoints #2748

merged 2 commits into from
Jun 13, 2016

Conversation

abarth
Copy link
Contributor

@abarth abarth commented Jun 12, 2016

This enables drawing polygons faster than using Paths.

Fixes flutter/flutter#92

This enables drawing polygons faster than using Paths.

Fixes flutter/flutter#92
@abarth
Copy link
Contributor Author

abarth commented Jun 12, 2016

@Hixie

@abarth abarth force-pushed the draw_points branch 2 times, most recently from 1894915 to 411ea5c Compare June 12, 2016 08:14
@jason-simmons
Copy link
Member

lgtm

// Draw the entire sequence of point as the vetices of a polygon.
polygon,
}

Copy link
Contributor

Choose a reason for hiding this comment

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

would be nice if VertexMode and PointMode discussed when you'd use each one
c.f. http://docs.flutter.io/flutter/dart-ui/BlurStyle-class.html

@Hixie
Copy link
Contributor

Hixie commented Jun 13, 2016

LGTM

Path.fillType
Path.addPolygon
Path.addPath
Path.extendWithPath
Path.transform

Fixes flutter/flutter#252
@abarth abarth merged commit 5f14c77 into flutter:master Jun 13, 2016
@abarth abarth deleted the draw_points branch June 13, 2016 18:42
@tvolkert
Copy link
Contributor

Getting the following, which seems related to this change:

../../sky/engine/core/painting/Canvas.cpp:349:14: error: out-of-line definition of 'drawVertices' does not match any declaration in 'blink::Canvas'
void Canvas::drawVertices(
             ^~~~~~~~~~~~
1 error generated.

@tvolkert
Copy link
Contributor

n/m - just picked up the fix

rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
New in this revision:

> git log --format="%C(auto) %h %s" b10966c..c00d4b4
 c00d4b4 Improve message in `pub outdated` when incompatible versions are found (flutter#2746)
 2a17762 outdated --mode=null-safety, resolvable constrained to null-safe vers… (flutter#2739)
 3ea2b83 Default language version (flutter#2748)
 7886546 Don't allow both --json and --transitive (flutter#2742)

Fixes dart-lang/sdk#44013

Change-Id: I0dc876c5e57777a4724d90662408f91910e2517c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/171720
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
Reviewed-by: Jonas Jensen <jonasfj@google.com>
Reviewed-by: Michael Thomsen <mit@google.com>
rhencke pushed a commit to rhencke/engine that referenced this pull request Dec 20, 2020
* Added a default language version of 2.7

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

Successfully merging this pull request may close these issues.

4 participants