Code Review

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Recent Closed Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
346900043	Issue 5399: Reuse find_context_above() for ly:context-find (Closed)	Dan Eble			1 day, 16 hours
369820043	Removes text that indicates windows support. (Closed)	alanmorales	tamasz		3 days, 2 hours
343350043	Addressing First Round Of Comments (Closed)	jamaalbwells			1 week, 1 day
363730043	Enhanced perfdiag command to record if running on GCE, GCE instance details, bucket location and st… (Closed)	Mike Schwartz	houglum		2 weeks, 4 days
305380043	Convert a bunch of C++ internals to degrees rather than radians (Closed)	dak	thomasmorley651, carl.d.sorensen, Carl	2	3 weeks
306710043	Avoid a few trivial lambda function wrappers (Closed)	dak	simon.albrecht	8	3 weeks
318010043	Correct the guns for hire list under "Community" (Closed)	dak			3 weeks
318200043	Rework the Preinit framework into something simpler (Closed)	dak	lemzwerg		3 weeks
314300043	Using eq? on numbers is undefined behavior (Closed)	dak	lemzwerg		3 weeks
315540043	HTML 4.01 requires "type" attribute for "script" element (Closed)	dak	Graham Percival		3 weeks
320170043	Redo send_stream_event in type-clean manner (Closed)	dak	lemzwerg		3 weeks
314500043	Avoid using void * as SCM (Closed)	dak	lemzwerg		3 weeks
316300043	Use explicit typecasts for Scm_variable->SCM (Closed)	dak	lemzwerg		3 weeks
314590043	Let \afterGrace start a Bottom context (Closed)	dak	lemzwerg		3 weeks
314490043	SCM/bool confusion in One_page_breaking::solve (Closed)	dak	pwm	1	3 weeks
316360043	Allow number as MarkEvent.label (Closed)	dak	lemzwerg		3 weeks
324140043	Allow defining markup commands in LilyPond syntax (Closed)	dak	thomasmorley651, lemzwerg	2	3 weeks
323940043	Various chain-assoc-get -> #:properties (Closed)	dak	thomasmorley651, carl.d.sorensen, Carl	3	3 weeks
314460043	Issue 5044: NR-de: fix Petrucci f clef example (Closed)	dak			3 weeks
317330043	NR: default markFormatter is format-mark-letters (Closed)	dak			3 weeks

1 - 20 Older ›