Code review comment for lp:~jamesh/libunity/search-changed

Revision history for this message
Michal Hruby (mhr3) wrote :

Looking good, just one simplification:

126 + uint tid = Timeout.add (500, () => { ml.quit (); return false; });
127 + ml.run ();

Could you please change this to assert(run_with_timeout(ml)) ?

review: Needs Fixing

« Back to merge proposal