[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fixes annotate spacing (issue 7313082)
|
From: |
dak |
|
Subject: |
Re: Fixes annotate spacing (issue 7313082) |
|
Date: |
Wed, 20 Feb 2013 15:32:22 +0000 |
On 2013/02/20 15:27:49, dak wrote:
https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm
File scm/skyline.scm (right):
https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm#newcode23
scm/skyline.scm:23: (define-public (skyline-pair::useable? skyp)
So why is this not non-empty-skyline-pair? or
skyline-pair-and-non-empty?
And if for some reason you really, really, want to start this off with
"skyline-pair::", then "skyline-pair::and-non-empty?" will perfectly
bring across checking for both being a skyline pair and non-empty.
https://codereview.appspot.com/7313082/