]> granicus.if.org Git - python/commit
Eliminated gcc -Wall complaints:
authorBarry Warsaw <barry@python.org>
Tue, 14 Jan 1997 17:45:08 +0000 (17:45 +0000)
committerBarry Warsaw <barry@python.org>
Tue, 14 Jan 1997 17:45:08 +0000 (17:45 +0000)
commit93d29b689509ddbd54ae8ea009c6731fba82cb81
treec1f53fd6d5f36d29a5ce31fb612d1796b177c90a
parent61a63e152d568d761e13988376c204dbc3675779
Eliminated gcc -Wall complaints:

    - Quieted gcc -Wall by removing unused local variables.

    - Added some choice parentheses around assignments in conditional
      tests.

    - Removed an unused (and seemingly unreachable) err label in
      load_short_binstring().

    - in Unpickler_load(), removed \. in string format.

    - init_stuff() was declared to return an int, but had these
      problems:

- it was returning NULL instead of 0 or 1 in some cases
- it was falling of the end of the routine without returning
  anything
- the call of init_stuff() in initcPickle() was never checking
  the return value anyway.

      I changed all this by returning 1 in the case of errors, 0 when
      no error occurred.  Then in initcPickle(), if init_stuff()
      returns non-zero, I call Py_FatalError().

Suppressing my urge to reformat according to Python coding standards!
:-)
Modules/cPickle.c