src/codec/SkCodecPriv.h - Issue 2390263002: Add SkColorSpaceXform to the public API

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: src/codec/SkCodecPriv.h

Issue 2390263002: Add SkColorSpaceXform to the public API (Closed)

Patch Set: Remove type on enum Created 4 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Index: src/codec/SkCodecPriv.h

diff --git a/src/codec/SkCodecPriv.h b/src/codec/SkCodecPriv.h

index b93def8790222d777e514aad8448e8265dbb2351..f210303598af22a1aff183f6c3e3a4744f65d530 100644

--- a/src/codec/SkCodecPriv.h

+++ b/src/codec/SkCodecPriv.h

@@ -176,8 +176,8 @@ static inline uint64_t get_color_table_fill_value(SkColorType dstColorType, SkAl

SkASSERT(colorXform);

uint64_t dstColor;

uint32_t srcColor = colorPtr[fillIndex];

- colorXform->apply(&dstColor, &srcColor, 1, select_xform_format(dstColorType),

- SkColorSpaceXform::kRGBA_8888_ColorFormat, alphaType);

+ SkAssertResult(colorXform->apply(select_xform_format(dstColorType), &dstColor,

+ SkColorSpaceXform::kRGBA_8888_ColorFormat, &srcColor, 1, alphaType));

return dstColor;

}

default:

« no previous file with comments | « include/core/SkColorSpaceXform.h ('k') | src/codec/SkJpegCodec.cpp » ('j') | no next file with comments »