diff options
| author | Guido van Rossum <guido@python.org> | 1998-07-07 22:19:27 (GMT) |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 1998-07-07 22:19:27 (GMT) |
| commit | eaa1ed6b42a188ff33362b47885b18993a2e0141 (patch) | |
| tree | 0f0f9480f858259a7ac3114e6196a32804cea1e3 | |
| parent | e32d1537f0a2629c5a9896b84bfef3b965c8eb99 (diff) | |
| download | cpython-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 '&'.
| -rw-r--r-- | Modules/almodule.c | 4 |
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 *) ¶m->value.i, pinfo->elementType); - break; case AL_INT64_ELEM: case AL_FIXED_ELEM: return python2elem(value, (void *) ¶m->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"); |