Davidlohr Bueso wrote:
> On Mon, 2011-11-14 at 02:48 +0100, Bernhard Voelker wrote:
> > [PATCH 10/15] prlimit: refactor code: separate modify and show clearly
> >
> > Struct prlimit now has a flag whether a limit has to be modified,
> > and a flag whether the limit has to be shown; names "modify" and
> > "show". Furthermore, move show_limits() to main().
> >
> > Signed-off-by: Bernhard Voelker <mail@xxxxxxxxxxxxxxxxxxx>
> > ---
> > sys-utils/prlimit.c | 37 +++++++++++++++++++------------------
> > 1 files changed, 19 insertions(+), 18 deletions(-)
> >
> > diff --git a/sys-utils/prlimit.c b/sys-utils/prlimit.c
> > index 4124faa..fc11a94 100644
> > --- a/sys-utils/prlimit.c
> > +++ b/sys-utils/prlimit.c
> > @@ -85,12 +85,12 @@ static struct prlimit_desc prlimit_desc[] =
> >
> > struct prlimit {
> > struct rlimit rlim;
> > - int found;
>
> Wait, doesn't patch 07 add this variable?
well, actually yes ... but effectively (to not break the order
in PRLIMIT_EMPTY_LIMIT), I decided to rename-and-move that
variable:
> > struct prlimit_desc *desc;
> > int modify;
> > + int show;
> > };
> >
> > -#define PRLIMIT_EMPTY_LIMIT {{ 0, 0, }, NULL, 0 }
> > +#define PRLIMIT_EMPTY_LIMIT {{ 0, 0, }, NULL, 0, 0 }
Bye,
Berny--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
