/srv/irclogs.ubuntu.com/2010/05/10/#launchpad-reviews.txt

=== barry` is now known as barry_
=== danilos changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [danilo] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== leonardr changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [danilo, leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== leonardr changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [danilo, leonardr, leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: - || queue: [leonardr, leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: leonardr || queue: [leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: leonardr || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bigjoolsintellectronica: ping14:25
intellectronicabigjools: yo14:26
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: stevenk || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bigjoolsintellectronica: just wondering if you were still reviewing Steve's branch?  I am going to approve it since I am familiar with what's going on, might save you some time :)14:27
intellectronicabigjools: great! was just starting on it, and it will be great not to have to go on :)14:28
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bigjoolsintellectronica: yeah  it's a typical Soyuz branch, you don't need that pain14:28
intellectronicabigjools: thanks!14:28
=== sinzui changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: - || queue: [sinzui] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
sinzuiintellectronica, I have a simple branch that I hope you have time to review14:29
intellectronicasinzui: just started on a branchlet from deryck but will look at yours as soon as i finish that14:29
sinzuithanks14:29
leonardrintellectronica, thanks for thos reviews14:35
intellectronicaleonardr: yer welcome14:38
intellectronicasinzui: is it the oopsless-repr branch?14:40
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
sinzuiintellectronica, it is14:40
intellectronicacool, on it14:40
intellectronicasinzui: is it not the case that encode('ASCII'... will raise an exception if you feed it stuff it can't encode?14:43
intellectronicasinzui: or is that taken care of completely by 'backslashreplace'?14:43
sinzuiintellectronica, yes.14:43
intellectronicasinzui: yes, the latter? :)14:44
sinzuiintellectronica,  'backslashreplace' replaces all the unicode possitions14:44
intellectronicacool14:44
sinzuiWe know the field is unicode, so if 'backslashreplace' fails, we have a deeper problem in the app14:45
intellectronicasinzui: r=me14:46
sinzuithanks14:46
=== jelmer_ is now known as jelmer
=== gary_poster_ is now known as gary_poster
marsintellectronica, room for a trivial cleanup branch?  https://code.edge.launchpad.net/~mars/launchpad/trivial-add-ec2-xxx/+merge/2492715:50
intellectronicamars: sure15:50
marsthanks15:50
marsintellectronica, FYI, then branch was reviewed earlier by Gary, but I made some post-review cleanups that need to be checked over.  The branch is still tiny though.15:51
intellectronicamars: cool, on it15:51
=== intellectronica changed the topic of #launchpad-reviews to: On call: intellectronica || reviewing: mars || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
intellectronicamars: r=me15:52
marsgreat, thanks Tom :)15:52
=== matsubara is now known as matsubara-lunch
=== gary_poster is now known as gary-lunch
=== matsubara-lunch is now known as matsubara
=== intellectronica changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== gary-lunch is now known as gary_poster
=== matsubara is now known as matsubara-afk
=== barry` is now known as barry_

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!