summaryrefslogtreecommitdiffstats
path: root/include/libimobiledevice/libimobiledevice.h
diff options
context:
space:
mode:
authorGravatar Martin Szulecki2014-10-05 16:51:03 +0200
committerGravatar Martin Szulecki2014-10-05 16:51:03 +0200
commit5072dea1373b7c4789a9ea1e65d05ea30acf41ed (patch)
tree51bdd0584d727080fab86acd28bd4b360b2333bd /include/libimobiledevice/libimobiledevice.h
parent5552fa0fcc7501d3206fb3d3a64c1d04062ec53b (diff)
downloadlibimobiledevice-5072dea1373b7c4789a9ea1e65d05ea30acf41ed.tar.gz
libimobiledevice-5072dea1373b7c4789a9ea1e65d05ea30acf41ed.tar.bz2
Convert int16_t macro error types into enum to improve debugging/type-checking
This simple change provides various benefits for developers like compile time errors, better auto-completition in editors and the ability of a debugger to show the constant instead of just the raw value. Thanks to Aaron Burghardt for the proposal.
Diffstat (limited to 'include/libimobiledevice/libimobiledevice.h')
-rw-r--r--include/libimobiledevice/libimobiledevice.h23
1 files changed, 10 insertions, 13 deletions
diff --git a/include/libimobiledevice/libimobiledevice.h b/include/libimobiledevice/libimobiledevice.h
index 3cbb96b..c91ec83 100644
--- a/include/libimobiledevice/libimobiledevice.h
+++ b/include/libimobiledevice/libimobiledevice.h
@@ -32,19 +32,16 @@ extern "C" {
#include <sys/stat.h>
#include <plist/plist.h>
-/** @name Error Codes */
-/*@{*/
-#define IDEVICE_E_SUCCESS 0
-#define IDEVICE_E_INVALID_ARG -1
-#define IDEVICE_E_UNKNOWN_ERROR -2
-#define IDEVICE_E_NO_DEVICE -3
-#define IDEVICE_E_NOT_ENOUGH_DATA -4
-#define IDEVICE_E_BAD_HEADER -5
-#define IDEVICE_E_SSL_ERROR -6
-/*@}*/
-
-/** Represents an error code. */
-typedef int16_t idevice_error_t;
+/** Error Codes */
+typedef enum {
+ IDEVICE_E_SUCCESS = 0,
+ IDEVICE_E_INVALID_ARG = -1,
+ IDEVICE_E_UNKNOWN_ERROR = -2,
+ IDEVICE_E_NO_DEVICE = -3,
+ IDEVICE_E_NOT_ENOUGH_DATA = -4,
+ IDEVICE_E_BAD_HEADER = -5,
+ IDEVICE_E_SSL_ERROR = -6
+} idevice_error_t;
typedef struct idevice_private idevice_private;
typedef idevice_private *idevice_t; /**< The device handle. */