I just hate it when I find that a "Draft" of mine was "Unpublished". This ...
8 years, 11 months ago
(2015-06-11 11:53:53 UTC)
#2
I just hate it when I find that a "Draft" of mine was "Unpublished". This is
really something that happens all the time when I am using Rietveld.
https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm
File scm/fret-diagrams.scm (right):
https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm#newcode266
scm/fret-diagrams.scm:266: (set! minfret (apply min (cons minfret (map last
barre-list)))))
Instead of (apply min (cons minfret (map last barre-list))) one can just write
(apply min minfret (map last barre-list))
since apply will just retain all arguments except the last one (which is
interpreted as a list of further arguments).
On 2015/06/11 11:53:53, dak wrote: > I just hate it when I find that a ...
8 years, 10 months ago
(2015-06-11 20:55:06 UTC)
#3
On 2015/06/11 11:53:53, dak wrote:
> I just hate it when I find that a "Draft" of mine was "Unpublished". This is
> really something that happens all the time when I am using Rietveld.
Thanks for uploading the patch and the review!
>
> https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm
> File scm/fret-diagrams.scm (right):
>
>
https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm#newcode266
> scm/fret-diagrams.scm:266: (set! minfret (apply min (cons minfret (map last
> barre-list)))))
> Instead of (apply min (cons minfret (map last barre-list))) one can just write
> (apply min minfret (map last barre-list))
> since apply will just retain all arguments except the last one (which is
> interpreted as a list of further arguments).
Done locally.
Any objections that I push it directly to staging or is a new review-circle
needed/wanted?
On 2015/06/11 20:55:06, thomasmorley651 wrote: > On 2015/06/11 11:53:53, dak wrote: > > I just ...
8 years, 10 months ago
(2015-06-11 21:09:46 UTC)
#4
On 2015/06/11 20:55:06, thomasmorley651 wrote:
> On 2015/06/11 11:53:53, dak wrote:
> > I just hate it when I find that a "Draft" of mine was "Unpublished". This
is
> > really something that happens all the time when I am using Rietveld.
>
> Thanks for uploading the patch and the review!
>
> >
> > https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm
> > File scm/fret-diagrams.scm (right):
> >
> >
>
https://codereview.appspot.com/247750043/diff/1/scm/fret-diagrams.scm#newcode266
> > scm/fret-diagrams.scm:266: (set! minfret (apply min (cons minfret (map last
> > barre-list)))))
> > Instead of (apply min (cons minfret (map last barre-list))) one can just
write
> > (apply min minfret (map last barre-list))
> > since apply will just retain all arguments except the last one (which is
> > interpreted as a list of further arguments).
>
>
> Done locally.
>
> Any objections that I push it directly to staging or is a new review-circle
> needed/wanted?
Just check locally that it runs on some file, then push.
Issue 247750043: fret-diagram with barre returns weird output
(Closed)
Created 8 years, 11 months ago by dak
Modified 8 years, 10 months ago
Reviewers: carl.d.sorensen_gmail.com, thomasmorley651
Base URL:
Comments: 1