summaryrefslogtreecommitdiffstats
path: root/Modules
diff options
context:
space:
mode:
authorGuido van Rossum <guido@python.org>1998-07-07 22:19:27 (GMT)
committerGuido van Rossum <guido@python.org>1998-07-07 22:19:27 (GMT)
commiteaa1ed6b42a188ff33362b47885b18993a2e0141 (patch)
tree0f0f9480f858259a7ac3114e6196a32804cea1e3 /Modules
parente32d1537f0a2629c5a9896b84bfef3b965c8eb99 (diff)
downloadcpython-eaa1ed6b42a188ff33362b47885b18993a2e0141.zip
cpython-eaa1ed6b42a188ff33362b47885b18993a2e0141.tar.gz
cpython-eaa1ed6b42a188ff33362b47885b18993a2e0141.tar.bz2
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 '&'.
Diffstat (limited to 'Modules')
-rw-r--r--Modules/almodule.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/Modules/almodule.c b/Modules/almodule.c
index a73e1be..7e399ea 100644
--- a/Modules/almodule.c
+++ b/Modules/almodule.c
@@ -289,12 +289,10 @@ python2param(int resource, ALpv *param, PyObject *value, ALparamInfo *pinfo)
case AL_ENUM_ELEM:
return python2elem(value, (void *) &param->value.i,
pinfo->elementType);
- break;
case AL_INT64_ELEM:
case AL_FIXED_ELEM:
return python2elem(value, (void *) &param->value.ll,
pinfo->elementType);
- break;
default:
PyErr_SetString(ErrorObject, "unknown element type");
return -1;
@@ -912,7 +910,7 @@ alp_ReadFrames(self, args)
int ch;
ALconfig c;
- if (!PyArg_ParseTuple(args, "i", framecount))
+ if (!PyArg_ParseTuple(args, "i", &framecount))
return NULL;
if (framecount < 0) {
PyErr_SetString(ErrorObject, "negative framecount");