Google Git
Sign in
skia / skia / 58a26a83628c42bb7de10ae6cd953aa823e9ee30
commit58a26a83628c42bb7de10ae6cd953aa823e9ee30[log] [tgz]
authorChris Dalton <csmartdalton@google.com>Mon Oct 19 19:51:28 2020 -0600
committerSkia Commit-Bot <skia-commit-bot@chromium.org>Fri Oct 23 22:09:46 2020 +0000
tree99210e927dd221e1db94850da34ac3c067990f95
parent8744a5c8202ef3c7dcc880d77e743528f3360037 [diff]
Fix the formula for fNumRadialSegmentsPerRadian in GrStrokePatchBuilder

GrStrokePatchBuilder had the wrong formula. This CL moves the formula to
GrStrokeTessellateOp and calculates it in one spot for the builder and
tessellator both. It also fixes a bug that was hiding behind this
formula error and updates some variable names.

Bug: skia:10419
Change-Id: I908d3960b16cac8dca0d40ff4116a3f5c5beed06
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/328416
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
7 files changed
tree: 99210e927dd221e1db94850da34ac3c067990f95
  1. animations/
  2. bazel/
  3. bench/
  4. bin/
  5. build/
  6. build_overrides/
  7. client_utils/
  8. demos.skia.org/
  9. dm/
  10. docker/
  11. docs/
  12. example/
  13. experimental/
  14. fuzz/
  15. gm/
  16. gn/
  17. include/
  18. infra/
  19. modules/
  20. platform_tools/
  21. resources/
  22. samplecode/
  23. site/
  24. specs/
  25. src/
  26. tests/
  27. third_party/
  28. tools/
  29. .bazelignore
  30. .clang-format
  31. .clang-tidy
  32. .gitignore
  33. .gn
  34. AUTHORS
  35. BUILD.bazel
  36. BUILD.gn
  37. codereview.settings
  38. CONTRIBUTING
  39. CQ_COMMITTERS
  40. DEPS
  41. go.mod
  42. go.sum
  43. LICENSE
  44. OWNERS
  45. PRESUBMIT.py
  46. public.bzl
  47. README
  48. README.chromium
  49. RELEASE_NOTES.txt
  50. whitespace.txt
  51. WORKSPACE.bazel