|
From: | dak |
Subject: | Re: Adds incipit section to NR (issue 108270043 by address@hidden) |
Date: | Sat, 16 Aug 2014 13:53:35 +0000 |
On 2014/08/16 13:12:56, PhilEHolmes wrote:
Please review further updates to this patch
Ok, this version does not offer any "Huh?" experiences apart from the primitive-eval which has sort of an easy to understand cause and no really convincing workaround. Have you checked that something useful happens when incipit-width is undefined? Half of the time I stare at that false-if-exception I am not really sure it will trigger at the right moment. https://codereview.appspot.com/108270043/
[Prev in Thread] | Current Thread | [Next in Thread] |