Merge lp://staging/~free.ekanayaka/txlongpoll/frontend-ajax-move-inline-functions-to-methods into lp://staging/txlongpoll

Proposed by Free Ekanayaka
Status: Merged
Approved by: Free Ekanayaka
Approved revision: 114
Merged at revision: 108
Proposed branch: lp://staging/~free.ekanayaka/txlongpoll/frontend-ajax-move-inline-functions-to-methods
Merge into: lp://staging/txlongpoll
Diff against target: 176 lines (+77/-67)
1 file modified
txlongpoll/frontend.py (+77/-67)
To merge this branch: bzr merge lp://staging/~free.ekanayaka/txlongpoll/frontend-ajax-move-inline-functions-to-methods
Reviewer Review Type Date Requested Status
Alberto Donato (community) Approve
Review via email: mp+298282@code.staging.launchpad.net

Commit message

Pure-refactoring branch, converting callbacks declared as inline functions to regular methods, for easier reading and isolation of local scope.

Description of the change

Pure-refactoring branch, converting callbacks declared as inline functions to regular methods, for easier reading and isolation of local scope.

To post a comment you must log in.
Revision history for this message
Alberto Donato (ack) wrote :

Looks good, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches