[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fixes problem with -dpreview cropping staff brackets (Issue 3386) (i
|
From: |
dak |
|
Subject: |
Re: Fixes problem with -dpreview cropping staff brackets (Issue 3386) (issue 9894043) |
|
Date: |
Fri, 31 May 2013 12:54:52 +0000 |
On 2013/05/31 12:29:46, PhilEHolmes wrote:
Please review.
Color me surprised. Patch set 3 from the original patch is called
"Patch Set 3 : Removes evil code from C++, replaces with happy defaults
in Scheme". Your partial revert only removes the "happy defaults in
Scheme" without reinstating the "evil code from C++". Now the question
is what the "evil code from C++" was supposed to be doing, and whether
merely removing the "happy defaults in Scheme" will cause problems.
Mike, are you reading this?
https://codereview.appspot.com/9894043/