Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba

2012-01-18 Thread David Graf
Review: Needs Fixing I tested it. Works really nice. I just have some little comments concerning the code: 1) Do we already have a bug entry for this: 336 // TODO: this was the initial implementation. This will have to change 337 -this-sleep_(1000); 338 +

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba

2012-01-18 Thread Gabriel Petrovay
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba

2012-01-18 Thread David Graf
Review: Approve Locks good! -- https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba

2012-01-17 Thread Gabriel Petrovay
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/88918 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :