From e3ed45b62fca4d08bfc9c2eaefc19ad72e637430 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Fri, 10 Jan 2014 12:07:48 +0100 Subject: [PATCH] These should probably use OBJRET_FILE_OK instead of true ...given the set of four OBJRET_FILE_* defines that apparently shall be returned from these functions. Also, the only place that looks at these return values is the switch in LoadDrawingObject that does nothing for both OBJECT_FILE_OK (== 0) and true (== 1) anyway. Change-Id: I04770071cd2f8df34328efce7aa1c533948a00af --- hwpfilter/source/drawing.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/hwpfilter/source/drawing.h b/hwpfilter/source/drawing.h index 904758a7922f..de8afcf96ac5 100644 --- a/hwpfilter/source/drawing.h +++ b/hwpfilter/source/drawing.h @@ -410,14 +410,14 @@ HWPDODefaultFunc(int , HWPDrawingObject * , int cmd, void *, int) { if (cmd == OBJFUNC_LOAD) return OBJRET_FILE_NO_PRIVATE_BLOCK; - return true; + return OBJRET_FILE_OK; } static int HWPDOLineFunc(int type, HWPDrawingObject * hdo, int cmd, void *argp, int argv) { - int ret = true; + int ret = OBJRET_FILE_OK; switch (cmd) { case OBJFUNC_LOAD: @@ -477,7 +477,7 @@ int cmd, void *argp, int argv) default: return HWPDODefaultFunc(type, hdo, cmd, argp, argv); } - return true; + return OBJRET_FILE_OK; } @@ -500,14 +500,14 @@ HWPDOArcFunc(int type, HWPDrawingObject * hdo, int cmd, void *argp, int argv) default: return HWPDODefaultFunc(type, hdo, cmd, argp, argv); } - return true; + return OBJRET_FILE_OK; } static int HWPDOArc2Func(int type, HWPDrawingObject * hdo, int cmd, void *argp, int argv) { - int ret = true; + int ret = OBJRET_FILE_OK; switch (cmd) { case OBJFUNC_LOAD: @@ -574,7 +574,7 @@ int cmd, void *argp, int argv) default: return HWPDODefaultFunc(type, hdo, cmd, argp, argv); } - return true; + return OBJRET_FILE_OK; } @@ -628,7 +628,7 @@ int cmd, void *argp, int argv) default: return HWPDODefaultFunc(type, hdo, cmd, argp, argv); } - return true; + return OBJRET_FILE_OK; }