[CRIU] Re: [PATCH 4/8] files: Print error if fcntl failed
Cyrill Gorcunov
gorcunov at openvz.org
Thu May 10 06:46:45 EDT 2012
On Thu, May 10, 2012 at 02:13:29PM +0400, Pavel Emelyanov wrote:
> On 05/05/2012 07:20 PM, Cyrill Gorcunov wrote:
> >
> > Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
> > ---
> > files.c | 19 +++++++++++++------
> > 1 files changed, 13 insertions(+), 6 deletions(-)
> >
>
> Applied, but I yet again remind you not to put two things in
> one patch. Changing old to ret just while you were at it wasn't
> very helpful :\
The patch changed the semantics of 'old' and leaving it named 'old'
not longer reflect what is used for, thus the change should be
done in one patch.
Cyrill
More information about the CRIU
mailing list