Documentation: C API comments in include/

Documentation: C API comments

Committed: https://skia.googlesource.com/skia/+/c9119060a0a2739f53b8e966e8fcd9ff58024580

[hal.canary, 2015-08-04 19:09:14]

halcanary@google.com changed reviewers:
+ reed@google.com

[hal.canary, 2015-08-04 19:09:15]

PTAL

[hal.canary, 2015-09-01 17:13:19]

ping

[reed1, 2015-09-01 17:35:15]

lgtm w/ nits

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_canvas.h
File include/c/sk_canvas.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_canvas.h#n...
include/c/sk_canvas.h:26: allocates an offscreen bitmap. All drawing calls are
directed
s/bitmap/surface

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_data.h
File include/c/sk_data.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_data.h#new...
include/c/sk_data.h:23: Returns a new sk_data_t by copying the specified source
data.
+= balance with a call to sk_data_unref(...)

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_paint.h
File include/c/sk_paint.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_paint.h#ne...
include/c/sk_paint.h:23: stroke miter : configurable at build-time, (usually
4.0f).
for the C api, this will always be 4.0

[hal.canary, 2015-09-01 17:39:53]

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_canvas.h
File include/c/sk_canvas.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_canvas.h#n...
include/c/sk_canvas.h:26: allocates an offscreen bitmap. All drawing calls are
directed
On 2015/09/01 17:35:15, reed1 wrote:
> s/bitmap/surface

Done.

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_data.h
File include/c/sk_data.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_data.h#new...
include/c/sk_data.h:23: Returns a new sk_data_t by copying the specified source
data.
On 2015/09/01 17:35:15, reed1 wrote:
> += balance with a call to sk_data_unref(...) 

Done.

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_paint.h
File include/c/sk_paint.h (right):

https://codereview.chromium.org/1271023002/diff/20001/include/c/sk_paint.h#ne...
include/c/sk_paint.h:23: stroke miter : configurable at build-time, (usually
4.0f).
On 2015/09/01 17:35:15, reed1 wrote:
> for the C api, this will always be 4.0

Done.

[hal.canary, 2015-09-01 17:39:55]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2015-09-01 17:39:56]

The patchset sent to the CQ was uploaded after l-g-t-m from reed@google.com
Link to the patchset: https://codereview.chromium.org/1271023002/#ps40001
(title: "2015-09-01 (Tuesday) 13:39:17 EDT")

[commit-bot: I haz the power, 2015-09-01 17:40:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1271023002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1271023002/40001

[commit-bot: I haz the power, 2015-09-01 17:45:12]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/c9119060a0a2739f53b8e966e8fcd9ff58024580