Mercurial > hg > orthanc-python
diff CodeAnalysis/FunctionBody.mustache @ 75:cbfc72a53970
refactoring calls to PythonLock::RaiseException()
| author | Sebastien Jodogne <s.jodogne@gmail.com> |
|---|---|
| date | Mon, 09 Aug 2021 16:59:59 +0200 |
| parents | 7ed502b17b8f |
| children | e7ff5efb100d |
line wrap: on
line diff
--- a/CodeAnalysis/FunctionBody.mustache Sat Jun 12 08:00:57 2021 +0200 +++ b/CodeAnalysis/FunctionBody.mustache Mon Aug 09 16:59:59 2021 +0200 @@ -5,7 +5,6 @@ {{#has_args}} if (!PyArg_ParseTuple(args, {{tuple_format}})) { - // TODO => RAISE : https://stackoverflow.com/questions/60832317 PyErr_SetString(PyExc_TypeError, "Bad types for the arguments ({{count_args}} arguments expected)"); return NULL; } @@ -34,9 +33,7 @@ {{#args}}{{release}}{{/args}} if (s.GetContent() == NULL) { - // TODO => RAISE : https://stackoverflow.com/questions/60832317 - //PythonLock::RaiseException(module, OrthancPluginErrorCode_InternalError); - PyErr_SetString(PyExc_ValueError, "Internal error"); + PythonLock::RaiseException(OrthancPluginErrorCode_InternalError); return NULL; } else @@ -62,9 +59,7 @@ } else { - // TODO => RAISE : https://stackoverflow.com/questions/60832317 - //PythonLock::RaiseException(module, code); - PyErr_SetString(PyExc_ValueError, "Internal error"); + PythonLock::RaiseException(code); return NULL; } {{/return_error}} @@ -74,9 +69,7 @@ {{#args}}{{release}}{{/args}} if (obj == NULL) { - // TODO => RAISE : https://stackoverflow.com/questions/60832317 - //PythonLock::RaiseException(module, OrthancPluginErrorCode_InternalError); - PyErr_SetString(PyExc_ValueError, "Internal error"); + PythonLock::RaiseException(OrthancPluginErrorCode_InternalError); return NULL; } else @@ -97,9 +90,7 @@ } else { - // TODO => RAISE : https://stackoverflow.com/questions/60832317 - //PythonLock::RaiseException(module, OrthancPluginErrorCode_InternalError); - PyErr_SetString(PyExc_ValueError, "Internal error"); + PythonLock::RaiseException(OrthancPluginErrorCode_InternalError); return NULL; } {{/return_bytes}}