← Back to team overview

zorba-coders team mailing list archive

Thread
Date

Re: [Merge] lp:~davidagraf/zorba/lastmodifiedfix into lp:zorba

To: David Graf <david.graf@xxxxxxxxxx>
From: Ghislain Fourny <ghislain.fourny@xxxxxxxxxx>
Date: Thu, 06 Sep 2012 09:45:31 -0000
In-reply-to: <20120904160817.2138.86127.launchpad@ackee.canonical.com>
Reply-to: mp+122712@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Nice catch! I would have used the constructor syntax rather than the assignment syntax for readability though (even though it has identical semantics - copy constructor call - in this case). But I leave the decision to you.
-- 
https://code.launchpad.net/~davidagraf/zorba/lastmodifiedfix/+merge/122712
Your team Zorba Coders is subscribed to branch lp:zorba.

References

[Merge] lp:~davidagraf/zorba/lastmodifiedfix into lp:zorba
From: David Graf, 2012-09-04