Code review comment for lp:~linuxjedi/drizzle/trunk-bug-689448

Revision history for this message
Brian Aker (brianaker) wrote :

Hi!

In this code you are working with a shared_ptr. This means that though you check it in one call, there is a chance it will have been changed by the time the second call is made. Either you need to make the call in just a single call, or you need to copy the shared_ptr locally and then make use of it from there.

Cheers,
   -Brina

« Back to merge proposal