Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit eaa1ed6

Browse files
committed
Fix some problems that a picky SGI compiler reported. Two were benign
(unreachable break after a return) but one was a real bug: ReadFrames() was getting a bogus framecount because of a missing '&'.
1 parent e32d153 commit eaa1ed6

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

Modules/almodule.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,12 +289,10 @@ python2param(int resource, ALpv *param, PyObject *value, ALparamInfo *pinfo)
289289
case AL_ENUM_ELEM:
290290
return python2elem(value, (void *) &param->value.i,
291291
pinfo->elementType);
292-
break;
293292
case AL_INT64_ELEM:
294293
case AL_FIXED_ELEM:
295294
return python2elem(value, (void *) &param->value.ll,
296295
pinfo->elementType);
297-
break;
298296
default:
299297
PyErr_SetString(ErrorObject, "unknown element type");
300298
return -1;
@@ -912,7 +910,7 @@ alp_ReadFrames(self, args)
912910
int ch;
913911
ALconfig c;
914912

915-
if (!PyArg_ParseTuple(args, "i", framecount))
913+
if (!PyArg_ParseTuple(args, "i", &framecount))
916914
return NULL;
917915
if (framecount < 0) {
918916
PyErr_SetString(ErrorObject, "negative framecount");

0 commit comments

Comments
 (0)