On Wed, May 31, 2017 at 06:58:07PM +0300, Dmitry V. Levin wrote:
> I'm very much in favour of this change, it makes the control flow easier
> to understand.  Technically, this patch looks correct, although I've spend
> a lot of my reviewing time to get to this conclusion.  Next time when you
> do refactoring, try making the change more obvious. :)

Okay. :)

> Do we need status variable along with *pstatus?

Probably yes, it saves a little extra dereferencing.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to