On Tue, 2016-04-12 at 08:54 +0200, Peter Krempa wrote:
On Mon, Apr 11, 2016 at 17:32:44 +0200, Andrea Bolognani wrote:
>
> ---
> There's one more potential instance on line 684
> (sc_require_whitespace_in_translation), but in that specific case
> I'm not 100% sure the whitespace is not supposed to be there.
I've tried to delete it and that check still works, so it looks it's not
necessary.
Yeah, since writing that comment I've convinced myself that
I could have safely deleted that whitespace as well. But I've
already pushed the patch, so it'll have to be in a separate
commit at some point in the future.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team