Re: [PATCH] pager: use an environment variable for pager_in_use
Johannes Schindelin wrote:
Show 6 quoted lines
>> Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
>>> +int pager_in_use = getenv("GIT_PAGER_IN_USE");
>> test.c:3: error: initializer element is not constant
> Sorry.
>
> Still, it feels wrong to use two system calls when you need none.getenv() is usually no system call, but processed in userland.
However, I also always get the feeling that using environment variables within one process to communicate state seems wrong, but many old unix tools do it this way.
cheers
simon
--
Serve - BSD +++ RENT this banner advert +++ ASCII Ribbon /"\
Work - Mac +++ space for low €€€ NOW!1 +++ Campaign \ /
Party Enjoy Relax | http://dragonflybsd.org Against HTML \
Dude 2c 2 the max ! http://golden-apple.biz Mail + News / \
Simon 'corecode' Schubert <corecode@fs.ei.tum.de>