Patches #54

Merged
mhameed merged 4 commits from patches into master 2012-01-08 14:50:43 -08:00
mhameed commented 2011-12-19 13:04:49 -08:00 (Migrated from github.com)

Hi Ryan, sorry for not getting back to you earlier.
A bit of further cleaning is provided in the additional commits.
Thanks.

Hi Ryan, sorry for not getting back to you earlier. A bit of further cleaning is provided in the additional commits. Thanks.
ryanmcgrath commented 2012-01-08 14:51:06 -08:00 (Migrated from github.com)

Hey, finally got around to merging this - thanks for the patch, I think people will be quite happy about this one! ;)

Hey, finally got around to merging this - thanks for the patch, I think people will be quite happy about this one! ;)
mhameed commented 2012-01-09 13:18:26 -08:00 (Migrated from github.com)

Sorry, looks like I got bitten by the way lambdas are implemented.

for key in ...:
self.dict[key] = ...

all assignments ended up pointing to the same endpoint, which was rather confusing.
I had to do a bit of rejuggling.
Please pull from patches for the fix.

This time it was tested :)

Sorry, looks like I got bitten by the way lambdas are implemented. for key in ...: self.**dict**[key] = ... all assignments ended up pointing to the same endpoint, which was rather confusing. I had to do a bit of rejuggling. Please pull from patches for the fix. This time it was tested :)
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#54
No description provided.