Note: I didn't remove C-style casts in SkPDFFont.cpp or SkPDFImage.cpp.
LGTM w/ comment. http://codereview.appspot.com/4887042/diff/1/src/pdf/SkPDFDevice.cpp File src/pdf/SkPDFDevice.cpp (right): http://codereview.appspot.com/4887042/diff/1/src/pdf/SkPDFDevice.cpp#newcode798 src/pdf/SkPDFDevice.cpp:798: sk_malloc_flags(numGlyphs * 2, SK_MALLOC_TEMP | SK_MALLOC_THROW)); Should this/these be in four more since Skia standard indent is 4?
http://codereview.appspot.com/4887042/diff/1/src/pdf/SkPDFDevice.cpp File src/pdf/SkPDFDevice.cpp (right): http://codereview.appspot.com/4887042/diff/1/src/pdf/SkPDFDevice.cpp#newcode798 src/pdf/SkPDFDevice.cpp:798: sk_malloc_flags(numGlyphs * 2, SK_MALLOC_TEMP | SK_MALLOC_THROW)); On 2011/08/15 20:56:17, Steve VanDeBogart wrote: > Should this/these be in four more since Skia standard indent is 4? We use 4 elsewhere. How does the new patch look to you?
LG