On Thu, Mar 8, 2012 at 6:30 PM, Chris Kühl <blixtra(a)gmail.com> wrote:
On Thu, Mar 8, 2012 at 5:55 PM, Patrick Ohly
<patrick.ohly(a)intel.com> wrote:
> On Thu, 2012-03-08 at 17:18 +0100, Chris Kühl wrote:
>> On Thu, Mar 8, 2012 at 4:43 PM, Patrick Ohly <patrick.ohly(a)intel.com>
wrote:
>> > On Wed, 2012-03-07 at 16:54 +0100, Chris Kühl wrote:
>> >> Now that you've merged for-master/new-master into master we are
>> >> rebasing onto master.
>> >
>> > How is that going?
>> >
>>
>> It's been rebased and we've squashed the changes into about a dozen
>> commits. The tests seem to give us the same results as before the
>> rebase. See the concurrent-sync-sessions-rebased-pre-cleanup branch
>> for the rebased and squashed changes. I'm putting the final touches on
>> the cleanup know.
>
> 057a19f18c600fe4158fd06087aed10b44f20c4e
>
<snip>
> Note that there is already a GetEnv() in util.h which returns an
empty
> std::string when getenv() returns NULL.
>
Thanks, I wasn't aware. I'll change it to use that.
Hmm, upon inspection I don't actually see this. I only see getEnv()
which returns a char*. I can simply use getenv, though.
Cheers,
Chris