Changeset 473


Ignore:
Timestamp:
Jan 2, 2013, 4:45:59 PM (7 years ago)
Author:
cito
Message:

Silence some compiler warnings, use consistent code style.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/module/pgmodule.c

    r471 r473  
    238238        if (err)
    239239        {
    240                 if (result) {
     240                if (result)
     241                {
    241242                        char *sqlstate = PQresultErrorField(result, PG_DIAG_SQLSTATE);
    242243                        str = sqlstate ? PyString_FromStringAndSize(sqlstate, 5) : NULL;
     
    507508                                                const int k = sizes[j];
    508509
    509                                                 if (align) {
     510                                                if (align)
     511                                                {
    510512                                                        sprintf(p, align == 'r' ?
    511513                                                                "%*s" : "%-*s", k,
     
    17731775        pgobject *self = (pgobject*) arg;
    17741776        PyObject *proc = self->notice_receiver;
    1775         if (proc && PyCallable_Check(proc)) {
     1777        if (proc && PyCallable_Check(proc))
     1778        {
    17761779                pgnoticeobject *notice = PyObject_NEW(pgnoticeobject, &PgNoticeType);
    17771780                PyObject *args, *ret;
     
    17841787                {
    17851788                        Py_INCREF(Py_None);
    1786                         notice = (pgnoticeobject *)Py_None;
     1789                        notice = (pgnoticeobject *)(void *)Py_None;
    17871790                }
    17881791                args = Py_BuildValue("(O)", notice);
     
    18061809                Py_END_ALLOW_THREADS
    18071810        }
    1808         if (self->notice_receiver) {
     1811        if (self->notice_receiver)
     1812        {
    18091813                Py_DECREF(self->notice_receiver);
    18101814        }
     
    25632567                                else
    25642568                                        *s = PyUnicode_AsEncodedString(obj, enc, "strict");
    2565                                 if (*s == NULL) {
     2569                                if (*s == NULL)
     2570                                {
    25662571                                        free(lparms); free(parms); free(str);
    25672572                                        PyErr_SetString(PyExc_UnicodeError, "query parameter"
    25682573                                                " could not be decoded (bad client encoding)");
    2569                                         while (i--) {
     2574                                        while (i--)
     2575                                        {
    25702576                                                if (*--s)
     2577                                                {
    25712578                                                        Py_DECREF(*s);
     2579                                                }
    25722580                                        }
    25732581                                        return NULL;
     
    25792587                        {
    25802588                                *s = PyObject_Str(obj);
    2581                                 if (*s == NULL) {
     2589                                if (*s == NULL)
     2590                                {
    25822591                                        free(lparms); free(parms); free(str);
    25832592                                        PyErr_SetString(PyExc_TypeError,
    25842593                                                "query parameter has no string representation");
    2585                                         while (i--) {
     2594                                        while (i--)
     2595                                        {
    25862596                                                if (*--s)
     2597                                                {
    25872598                                                        Py_DECREF(*s);
     2599                                                }
    25882600                                        }
    25892601                                        return NULL;
     
    26032615                {
    26042616                        if (*s)
     2617                        {
    26052618                                Py_DECREF(*s);
     2619                        }
    26062620                }
    26072621                free(str);
     
    30853099
    30863100static PyObject *
    3087 pg_escape_literal(pgobject *self, PyObject *args) {
     3101pg_escape_literal(pgobject *self, PyObject *args)
     3102{
    30883103        char *str; /* our string argument */
    30893104        int str_length; /* length of string */
     
    31073122
    31083123static PyObject *
    3109 pg_escape_identifier(pgobject *self, PyObject *args) {
     3124pg_escape_identifier(pgobject *self, PyObject *args)
     3125{
    31103126        char *str; /* our string argument */
    31113127        int str_length; /* length of string */
     
    31313147
    31323148static PyObject *
    3133 pg_escape_string(pgobject *self, PyObject *args) {
     3149pg_escape_string(pgobject *self, PyObject *args)
     3150{
    31343151        char *from; /* our string argument */
    31353152        char *to=NULL; /* the result */
     
    31413158                return NULL;
    31423159        to_length = 2*from_length + 1;
    3143         if (to_length < from_length) { /* overflow */
     3160        if (to_length < from_length) /* overflow */
     3161        {
    31443162                to_length = from_length;
    31453163                from_length = (from_length - 1)/2;
     
    31613179
    31623180static PyObject *
    3163 pg_escape_bytea(pgobject *self, PyObject *args) {
     3181pg_escape_bytea(pgobject *self, PyObject *args)
     3182{
    31643183        unsigned char *from; /* our string argument */
    31653184        unsigned char *to; /* the result */
     
    36003619
    36013620static PyObject *
    3602 escape_string(PyObject *self, PyObject *args) {
     3621escape_string(PyObject *self, PyObject *args)
     3622{
    36033623        char *from; /* our string argument */
    36043624        char *to=NULL; /* the result */
     
    36103630                return NULL;
    36113631        to_length = 2*from_length + 1;
    3612         if (to_length < from_length) { /* overflow */
     3632        if (to_length < from_length) /* overflow */
     3633        {
    36133634                to_length = from_length;
    36143635                from_length = (from_length - 1)/2;
     
    36293650
    36303651static PyObject *
    3631 escape_bytea(PyObject *self, PyObject *args) {
     3652escape_bytea(PyObject *self, PyObject *args)
     3653{
    36323654        unsigned char *from; /* our string argument */
    36333655        unsigned char *to; /* the result */
     
    36523674
    36533675static PyObject
    3654 *unescape_bytea(PyObject *self, PyObject *args) {
     3676*unescape_bytea(PyObject *self, PyObject *args)
     3677{
    36553678        unsigned char *from; /* our string argument */
    36563679        unsigned char *to; /* the result */
     
    37043727set_namedresult(PyObject *self, PyObject *args)
    37053728{
    3706     PyObject *ret = NULL;
    3707     PyObject *func;
    3708 
    3709     if (PyArg_ParseTuple(args, "O", &func))
    3710     {
    3711         if (PyCallable_Check(func)) {
    3712             Py_XINCREF(func); Py_XDECREF(namedresult); namedresult = func;
    3713             Py_INCREF(Py_None); ret = Py_None;
    3714         }
    3715         else
    3716             PyErr_SetString(PyExc_TypeError, "parameter must be callable");
    3717     }
    3718     return ret;
     3729        PyObject *ret = NULL;
     3730        PyObject *func;
     3731
     3732        if (PyArg_ParseTuple(args, "O", &func))
     3733        {
     3734                if (PyCallable_Check(func))
     3735                {
     3736                        Py_XINCREF(func); Py_XDECREF(namedresult); namedresult = func;
     3737                        Py_INCREF(Py_None); ret = Py_None;
     3738                }
     3739                else
     3740                        PyErr_SetString(PyExc_TypeError, "parameter must be callable");
     3741        }
     3742        return ret;
    37193743}
    37203744
Note: See TracChangeset for help on using the changeset viewer.