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
This commit is contained in:
Stephan Bergmann 2014-01-10 12:07:48 +01:00
parent b6d5125004
commit e3ed45b62f

View file

@ -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;
}