first draft of error checking / reporting API

include/core/SkError.h

Index: include/core/SkError.h
diff --git a/include/core/SkError.h b/include/core/SkError.h
new file mode 100644
index 0000000000000000000000000000000000000000..2b4e9789dbdaac8bbcc41025aad426cfa588a51a
--- /dev/null
+++ b/include/core/SkError.h
@@ -0,0 +1,28 @@
+/*
+ * Copyright 2013 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+ 
+#ifndef SkError_DEFINED
+#define SkError_DEFINED
+
+typedef enum {

[reed1, 2013/04/08 19:47:31]

I think all of our other enums are just

enum SkError {
...
};

[reed1, 2013/04/08 19:47:31]

Skia's other enums are named in reverse order from these.

e.g.

kNone_SkError,
kInvalidArgument_SkError,
...

[humper, 2013/04/08 20:10:55]

Okay, I'll turn these around.

On 2013/04/08 19:47:31, reed1 wrote:

+    kError_NoError=0,
+    kError_InvalidArgument,
+    kError_InvalidOperation,
+    kError_InvalidHandle,

[reed1, 2013/04/08 19:47:31]

What is a 'Handle' in the Skia world?

[humper, 2013/04/08 20:10:55]

Done.

+    kError_InvalidPaint,

[reed1, 2013/04/08 19:47:31]

What is an invalid paint?

+    kError_OutOfMemory,
+    kError_ParseError
+} SkError;
+
+SkError SkGetLastError();

[reed1, 2013/04/08 19:47:31]

Need some dox for each of these, including that they are per-thread.

[humper, 2013/04/08 20:10:55]

Done.

+void SkClearLastError();
+typedef void (*SkErrorCallbackFunction)(SkError, void *);
+
+void SkSetErrorCallback(SkErrorCallbackFunction cb, void *context);
+const char *SkGetLastErrorString();
+
+#endif /* SkError_DEFINED */