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

[SourceForge.net]

Patches item #790443, was opened at 2003-08-18 04:59
Message generated for change (Comment added) made by rhettinger
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: Closed
>Resolution: Fixed
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.

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

>Comment By: Raymond Hettinger (rhettinger)
Date: 2003-09-01 18:29

Message:
Logged In: YES 
user_id=80475

The double quoting part seems fine to me.  If it is driving 
you nuts, re-open this patch and assign to Barry Warsaw who 
wrote those lines.

The part about SafeConfigParser is reasonable.  Am updating 
the files now.

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

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