[coreboot-gerrit] New patch to review for coreboot: bd53d77 util/cbfstool: enum'ify return error codes

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Sat Apr 12 07:13:55 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5489

-gerrit

commit bd53d77c6c93502b0efa64cfc54748c3bf4147f8
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat Apr 12 15:10:36 2014 +1000

    util/cbfstool: enum'ify return error codes
    
    NOTFORMERGE
    
    work on `standardising` error return values with enum's.
    
    Change-Id: Iec74a766a3706bc45a0d7580c0f60227426a4c85
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 util/cbfstool/lzma/C/LzmaDec.c |  2 +-
 util/cbfstool/lzma/C/Types.h   | 39 ++++++++++++++++++++++-----------------
 2 files changed, 23 insertions(+), 18 deletions(-)

diff --git a/util/cbfstool/lzma/C/LzmaDec.c b/util/cbfstool/lzma/C/LzmaDec.c
index e208877..4393478 100644
--- a/util/cbfstool/lzma/C/LzmaDec.c
+++ b/util/cbfstool/lzma/C/LzmaDec.c
@@ -122,7 +122,7 @@ Out:
     = kMatchSpecLenStart + 2 : State Init Marker
 */
 
-static int LzmaDec_DecodeReal(struct CLzmaDec *p, size_t limit_parm, const uint8_t *bufLimit)
+static sz_error_t LzmaDec_DecodeReal(struct CLzmaDec *p, size_t limit_parm, const uint8_t *bufLimit)
 {
   CLzmaProb *probs = p->probs;
 
diff --git a/util/cbfstool/lzma/C/Types.h b/util/cbfstool/lzma/C/Types.h
index ce98ab5..89f85de 100644
--- a/util/cbfstool/lzma/C/Types.h
+++ b/util/cbfstool/lzma/C/Types.h
@@ -8,24 +8,29 @@
 #include <stdint.h>
 #include <stdbool.h>
 
+typedef enum
+{
+  SZ_OK = 0,
+  SZ_ERROR_DATA,
+  SZ_ERROR_MEM,
+  SZ_ERROR_CRC,
+  SZ_ERROR_UNSUPPORTED,
+  SZ_ERROR_PARAM,
+  SZ_ERROR_INPUT_EOF,
+  SZ_ERROR_OUTPUT_EOF,
+  SZ_ERROR_READ,
+  SZ_ERROR_WRITE,
+  SZ_ERROR_PROGRESS,
+  SZ_ERROR_FAIL,
+  SZ_ERROR_THREAD,
+
+  SZ_ERROR_ARCHIVE = 16,
+  SZ_ERROR_NO_ARCHIVE,
+
+  /* terminating elm to loop over enum */
+  SZ_LAST_ERROR
+} sz_error_t;
 
-#define SZ_OK 0
-
-#define SZ_ERROR_DATA 1
-#define SZ_ERROR_MEM 2
-#define SZ_ERROR_CRC 3
-#define SZ_ERROR_UNSUPPORTED 4
-#define SZ_ERROR_PARAM 5
-#define SZ_ERROR_INPUT_EOF 6
-#define SZ_ERROR_OUTPUT_EOF 7
-#define SZ_ERROR_READ 8
-#define SZ_ERROR_WRITE 9
-#define SZ_ERROR_PROGRESS 10
-#define SZ_ERROR_FAIL 11
-#define SZ_ERROR_THREAD 12
-
-#define SZ_ERROR_ARCHIVE 16
-#define SZ_ERROR_NO_ARCHIVE 17
 
 typedef int SRes;
 typedef int WRes; /* This was DWORD for _WIN32. That's uint32_t */



More information about the coreboot-gerrit mailing list