Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1718)

Issue 6331050: export flags on the state of the world when a picture was serialized. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 4 months ago by reed1
Modified:
12 years, 4 months ago
Reviewers:
DerekS, bsalomon
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

export flags on the state of the world when a picture was serialized. e.g. don't read/write functionptrs in that case (sizeof may be different for one) Committed: https://code.google.com/p/skia/source/detail?r=4318

Patch Set 1 #

Patch Set 2 : #

Total comments: 1

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -9 lines) Patch
M include/core/SkFlattenable.h View 3 chunks +16 lines, -1 line 0 comments Download
M include/core/SkPicture.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/core/SkFlattenable.cpp View 1 1 chunk +10 lines, -0 lines 0 comments Download
M src/core/SkPicture.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M src/core/SkPicturePlayback.cpp View 3 chunks +26 lines, -0 lines 0 comments Download
M src/core/SkXfermode.cpp View 1 chunk +5 lines, -7 lines 0 comments Download

Messages

Total messages: 4
reed1
12 years, 4 months ago (2012-06-22 18:34:35 UTC) #1
reed1
patch #2 inits the flags, but those should be set explicitly by the caller if ...
12 years, 4 months ago (2012-06-22 19:11:05 UTC) #2
bsalomon
https://codereview.appspot.com/6331050/diff/3001/src/core/SkXfermode.cpp File src/core/SkXfermode.cpp (left): https://codereview.appspot.com/6331050/diff/3001/src/core/SkXfermode.cpp#oldcode693 src/core/SkXfermode.cpp:693: buffer.writeFunctionPtr(NULL); Should writeFunctionPtr just check x-process and do this ...
12 years, 4 months ago (2012-06-22 19:14:07 UTC) #3
bsalomon
12 years, 4 months ago (2012-06-22 20:05:22 UTC) #4
LGTM

On 2012/06/22 19:14:07, bsalomon wrote:
> https://codereview.appspot.com/6331050/diff/3001/src/core/SkXfermode.cpp
> File src/core/SkXfermode.cpp (left):
> 
>
https://codereview.appspot.com/6331050/diff/3001/src/core/SkXfermode.cpp#oldc...
> src/core/SkXfermode.cpp:693: buffer.writeFunctionPtr(NULL);
> Should writeFunctionPtr just check x-process and do this for us (and
> readFunctionPtr just returns NULL)? Or are there cases where a function
pointer
> is not really a function pointer?
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b