[Patches] [ python-Patches-790443 ] add SafeConfigParser to __all__

[SourceForge.net]

Patches item #790443, was opened at 2003-08-18 18:59
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=790443&group_id=5470

Category: Library (Lib)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: George Yoshida (quiver)
Assigned to: Nobody/Anonymous (nobody)
Summary: add SafeConfigParser to __all__

Initial Comment:
Starting with Python 2.3, SafeConfigParser is preferred 
over ConfigParser.
http://www.python.org/doc/current/lib/module-
ConfigParser.html#l2h-1231
So why not add SafeConfigParser to __all__?

There's another fix.
RawConfigParser._read has a strange code.

  # allow empty values
  if optval == '""':
      optval = ''

This if-condition is evaluated true if and only if 
name=value pair is as follows:
  name = ""
and this code converts the value to an empty string, ''.

'""' is a string of two double quotes and '' is an empty 
string.
They're totally different.

I think this part is redundant.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=790443&group_id=5470