Menu

#336 Incorrect warning for function declared after use

0.8.18
open
nobody
None
5
2014-10-24
2014-10-24
No

Given:

def split_value( value, delim ):
    return [ i.strip() for i in value.split( delim ) ]

def format_user_info( name, roles ):
    return name + ";" + ':'.join( split_value( roles, ':' ) ) # <-- line 7

print format_user_info( "foo", ' a : b : c ' )

with pychecker 0.8.18, I get:

~~~~~~
Processing module x (/tmp/x.py)...
foo;a:b:c

Warnings...

/tmp/x.py:7: Using the return value from (split_value) which is always None
~~~~~~~

If the functions are declared in the other order, there is no warning.

Discussion


Log in to post a comment.