Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#19 Use Watcher objects rather than boolean on read operations

3.0.0
closed
Benjamin Reed
client (1)
5
2008-06-10
2008-05-12
Benjamin Reed
No

Rather than passing a boolean watch as a parameter to the read operations, it would be nice if the caller passed a watch object. The client library would keep track of which objects to invoke on a watch callback, so the protocol wouldn't change, but the client would be doing a bit more work.

Discussion

1 2 > >> (Page 1 of 2)
  • Benjamin Reed
    Benjamin Reed
    2008-05-12

    Logged In: YES
    user_id=154690
    Originator: YES

    Duplicates 1937078

     
  • Benjamin Reed
    Benjamin Reed
    2008-05-12

    • status: open --> wont-fix
     
  • Benjamin Reed
    Benjamin Reed
    2008-05-27

    Logged In: YES
    user_id=154690
    Originator: YES

    Whoops 1937078 is the patch that fixes this bug. Leaving open.

     
  • Benjamin Reed
    Benjamin Reed
    2008-05-27

    • assigned_to: nobody --> phunt
    • labels: 1037121 --> client
    • milestone: --> 3.0.0
    • status: wont-fix --> open
     
  • Patrick Hunt
    Patrick Hunt
    2008-05-29

    • assigned_to: phunt --> breed
     
  • Patrick Hunt
    Patrick Hunt
    2008-05-29

    Logged In: YES
    user_id=12853
    Originator: NO

    I have implemented the java version and submitted a patch. Ben can you re-assign to someone who can knock out the c code? I could do it but my c is pretty rusty.

     
  • Benjamin Reed
    Benjamin Reed
    2008-06-06

    Logged In: YES
    user_id=154690
    Originator: YES

    Fixed with:
    [ 1937078 ] Passing a watch object to read requests

     
  • Benjamin Reed
    Benjamin Reed
    2008-06-06

    • status: open --> closed
     
  • Benjamin Reed
    Benjamin Reed
    2008-06-06

    • status: closed --> open
     
  • Benjamin Reed
    Benjamin Reed
    2008-06-06

    Logged In: YES
    user_id=154690
    Originator: YES

    Oops, we still need the C client....

     
1 2 > >> (Page 1 of 2)