a008b0fa8b
To my surprise, this even works with homegrown smart pointers (such as SkTLazy). https://clang.llvm.org/extra/clang-tidy/checks/readability-redundant-smartptr-get.html Find and remove redundant calls to smart pointer’s .get() method. Examples: ptr.get()->Foo() ==> ptr->Foo() *ptr.get() ==> *ptr *ptr->get() ==> **ptr if (ptr.get() == nullptr) ... => if (ptr == nullptr) ... Change-Id: I8ff541e0229656b4d8e875c8053a7e6138302547 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/310976 Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: Mike Klein <mtklein@google.com> Reviewed-by: Mike Klein <mtklein@google.com>
29 lines
953 B
C++
29 lines
953 B
C++
/*
|
|
* Copyright 2016 Google Inc.
|
|
*
|
|
* Use of this source code is governed by a BSD-style license that can be
|
|
* found in the LICENSE file.
|
|
*/
|
|
|
|
#include "experimental/svg/model/SkSVGRenderContext.h"
|
|
#include "experimental/svg/model/SkSVGShape.h"
|
|
|
|
SkSVGShape::SkSVGShape(SkSVGTag t) : INHERITED(t) {}
|
|
|
|
void SkSVGShape::onRender(const SkSVGRenderContext& ctx) const {
|
|
const auto fillType = ctx.presentationContext().fInherited.fFillRule->asFillType();
|
|
|
|
// TODO: this approach forces duplicate geometry resolution in onDraw(); refactor to avoid.
|
|
if (const SkPaint* fillPaint = ctx.fillPaint()) {
|
|
this->onDraw(ctx.canvas(), ctx.lengthContext(), *fillPaint, fillType);
|
|
}
|
|
|
|
if (const SkPaint* strokePaint = ctx.strokePaint()) {
|
|
this->onDraw(ctx.canvas(), ctx.lengthContext(), *strokePaint, fillType);
|
|
}
|
|
}
|
|
|
|
void SkSVGShape::appendChild(sk_sp<SkSVGNode>) {
|
|
SkDebugf("cannot append child nodes to an SVG shape.\n");
|
|
}
|