Skip to content
Snippets Groups Projects
Commit 2ce71fa9 authored by Thomas Jahns's avatar Thomas Jahns :cartwheel:
Browse files

Simplify control flow in tableInqParName.

parent edebd854
No related branches found
No related tags found
No related merge requests found
...@@ -825,27 +825,21 @@ int tableInqParCode(int tableID, char *varname, int *code) ...@@ -825,27 +825,21 @@ int tableInqParCode(int tableID, char *varname, int *code)
int tableInqParName(int tableID, int code, char *varname) int tableInqParName(int tableID, int code, char *varname)
{ {
int item, npars; int err = 1;
int err = 0;
npars = parTable[tableID].npars;
if ( tableID == UNDEFID ) if ( tableID != UNDEFID )
{
err = 1;
}
else
{ {
for ( item = 0; item < npars; item++ ) int npars = parTable[tableID].npars;
for ( int item = 0; item < npars; item++ )
{ {
if ( parTable[tableID].pars[item].id == code ) if ( parTable[tableID].pars[item].id == code )
{ {
if ( parTable[tableID].pars[item].name ) if ( parTable[tableID].pars[item].name )
strcpy(varname, parTable[tableID].pars[item].name); //FIXME: This may overrun the supplied buffer! strcpy(varname, parTable[tableID].pars[item].name); //FIXME: This may overrun the supplied buffer!
err = 0;
break; break;
} }
} }
if ( item == npars ) err = 1;
} }
return (err); return (err);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment