[ python-Bugs-1723875 ] inspect.formatargspec last argument ignored

[SourceForge.net]

Bugs item #1723875, was opened at 2007-05-23 00:46
Message generated for change (Comment added) made by patrickcd
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=1723875&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Python Library
Group: Python 2.5
>Status: Closed
>Resolution: Invalid
>Priority: 1
Private: No
Submitted By: Patrick Dobbs (patrickcd)
Assigned to: Nobody/Anonymous (nobody)
Summary: inspect.formatargspec last argument ignored

Initial Comment:
from: patrick at laculine.com

The last argument of inspect.formatargspect() is a function to control the way the entire returned string is formatted:

 ...join=joinseq):

joinseq being a function at line 757 in inspect.py

However, this argument is ignored and joinseq is not called.

The line in question is inspect.py 794 it was:

    return '(' + string.join(specs, ', ') + ')'

The function seems to work as advertised if this line is replaced by: 

    return join(specs)

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

>Comment By: Patrick Dobbs (patrickcd)
Date: 2007-06-01 22:42

Message:
Logged In: YES 
user_id=314154
Originator: YES

On writing a test case I realised that that this bug report is invalid.
The ninth argument "join" is specified as determining how the arguments are
formatted, not how the entire returned string is formatted. It works
perfectly well. 

I can submit a lame test case which proves this bug report is bogus, but
it seems better to retire gracefully.. "TAXI!!"


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

Comment By: Raghuram Devarakonda (draghuram)
Date: 2007-05-31 15:41

Message:
Logged In: YES 
user_id=984087
Originator: NO


A simple look at the source proves your point. BTW, the same problem
exists for formatargvalues(). Do you think you can work up a patch? The
code change itself would be simple but the patch would require a test case.

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

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