From: Michael <barra_cuda@katamail.com>
Date: Sun, 20 May 2007 14:41:20 GMT
Subject: Re: [QGit PATCH] Remove most ASSERT warnings in Git::setStatus
Message-ID: <200705201641.20305.barra_cuda@katamail.com>
In-Reply-To: <e5bfff550705200723i7b4e21ebi6a2c51d66659b388@mail.gmail.com>

"Marco Costalba" <mcostalba@gmail.com>:
> So I rather would prefer something like
> 
>   line.section('\t', 0, 0).section(' ', -1, -1).left(1)
> 
> because we could have more then one file separated by a tab, so
> 
>  line.section('\t', -2, -2).right(1)
> 
> it seems to me a little bit fragile. What do you think?

You're right.

> Also I don't understand why you consider the right most (right(1)),
> instead of the left most character as the status.

Only because it was simpler AND because I didn't know it was wrong.