Code review comment for lp://staging/~larsu/telepathy-indicator/fix-warnings

Revision history for this message
Charles Kerr (charlesk) wrote :

Looks fine and seems to work. Approve.

On style points, I wish this had been a NullObject rather than a NULL pointer test -- there are so many more logic branches in the code now. This is going to hurt the coverage stats. Oh wait... ;-)

review: Approve

« Back to merge proposal