id summary reporter owner description type status priority milestone component version severity resolution keywords cc distribution status_field 1211 let the logger do the formatting sascha_silbe tomeu "With the attached patch we delegate the string formatting to the logger instead of doing it ourselves. Advantages: a) It's more robust: Constructs like 'debug(""foo %r bar"" % x)' are quite common in our code and will break if x is a tuple. 'debug(""foo %r bar"", x)' will work regardless of the type of x. b) formatting only happens if we actually log something. I haven't measured the impact of b) because a) is much more important to me (happened a lot of times during the object_id => (tree_id, version_id) conversion). At least one project has asked for pylint to flag these occurences and gave performance as a reason for the change, though (so they mave have measured it). " defect closed Unspecified by Maintainer Sugar Unspecified Unspecified fixed r+ Unspecified New