Quoting Sukadev Bhattiprolu ([email protected]): > [email protected] [[email protected]] wrote: > | From: Serge E. Hallyn <[email protected]> > | > | Matt Helsley originally suggested this to avoid having two > | format strings. This is not bisect-safe and therefore not > | even compile-tested. Every call to ckpt_write_err must be > | updated to use a single format. > > It maybe easier to review this patch if this is broken up into smaller > patches: > > - move the code to new place > - leave the fmt0 parameter to ckpt_generate_fmt() but ensure it > is unused. > - finally remove the unused parameters from ckpt_generate_fmt() > and the callers.
That's going a bit far... I did leave the moving of the code in a separate patch up until this last posting, but the new and old versions of ckpt_generate_fmt() don't look alike enough to be able to compare side by side anyway imo. One thing which would help however imo would be if I keep a non-rebased tagged tree alongside the rebased tree, so you can see what changes I'm making over the last posting. I'll do that as of today. -serge _______________________________________________ Containers mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/containers _______________________________________________ Devel mailing list [email protected] https://openvz.org/mailman/listinfo/devel
