PathApplierFunction should take a reference to a PathElement
Created attachment 257610 [details] Patch
Created attachment 257624 [details] Patch
Comment on attachment 257624 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257624&action=review Obviously, we need to rebase this patch before landing. > Source/WebCore/platform/graphics/cg/PathCG.cpp:383 > +static void CGPathApplierToPathApplier(void *info, const CGPathElement* element) We should also fix up the position of the '*' in the first argument.
https://trac.webkit.org/r187492
Comment on attachment 257624 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257624&action=review > Source/WebCore/platform/graphics/Path.h:83 > + typedef void (*PathApplierFunction)(void* info, const PathElement&); We should come back here and change this to be: typedef std::function<void (const PathElement&)> PathApplierFunction; Cleaner to use std::function instead of void*.
> We should come back here and change this to be: > > typedef std::function<void (const PathElement&)> PathApplierFunction; > > Cleaner to use std::function instead of void*. Doing so via bug 147368