Changes between Initial Version and Version 1 of Ticket #2507


Ignore:
Timestamp:
2015-09-17T08:20:00Z (9 years ago)
Author:
daira
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #2507

    • Property Status changed from new to assigned

  • Ticket #2507 – Description

    initial v1  
    1 [09:05] <daira1> meejah: we don't use inlineCallbacks in Tahoe mainly due to inertia -- the vast majority of the codebase was written without them...
    2 [09:07] <daira1> I had some minor technical reservations about them (and about mixing the two styles) but I should probably reconsider whether those are really justified
    3 [09:09] <daira1> currently our only use of them is in the tests for some of the cloud backend service providers (on a dev branch) that Leif wrote
    4 [09:10] <daira1> I think there's no objection to using them more generally in tests
     1<daira> meejah: we don't use inlineCallbacks in Tahoe mainly due to inertia -- the vast majority of the codebase was written without them...
     2<daira> I had some minor technical reservations about them (and about mixing the two styles) but I should probably reconsider whether those are really justified
     3<daira> currently our only use of them is in the tests for some of the cloud backend service providers (on a dev branch) that Leif wrote
     4<daira> I think there's no objection to using them more generally in tests
    55
    66For this ticket, understand pros and cons of using @inlineCallbacks and add a section to the CodingStandards with guidelines for their use. (This might be very short if no guidelines are needed!)