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
323420043	Issue 4678: Fix spaces in metronome mark (Closed)	Ebe123	dak		3 hours, 40 minutes
323340043	ticket:13285 Adding NumberingSystem constructor methods to DecimalFormatSymbols (J and C). (Closed)	sffc	markus.icu	26	10 hours, 58 minutes
316440043	Web: Addition to 'What we wrote' section (Closed)	pkx166h			14 hours, 39 minutes
297130043	timed_traces (other files) (Closed)	kirill.sc	marcus.falgert		1 day, 3 hours
310630043	update for review (Closed)	kirill.sc	lorenzo.corneo		1 day, 3 hours
315740043	Code Review: Current path classifier (Closed)	kirill.sc	lorenzo.corneo	32	1 day, 3 hours
312170043	Adding YCSB timestamping for tectonic (Closed)	kirill.sc	lorenzo.corneo	6	1 day, 3 hours
305640043	update test (Closed)	kirill.sc			1 day, 3 hours
305490043	sample add (Closed)	kirill.sc			1 day, 3 hours
323400043	ticket:13274 Break Iterator Test, cleanups & Java porting. (Closed)	andy.heninger	nrunge	2	1 day, 11 hours
325250043	[plaso] Changes to handle preferred time zone when not set #929 (Closed)	Joachim Metz	jberggren		1 day, 15 hours
330030043	[plaso] Changes to troubleshoot issues with respawn (Closed)	Joachim Metz	aaronp		1 day, 15 hours
323410043	Fix issue 5110: Don't use fixed color for LyricHyphens (Closed)	pkx166h			2 days, 4 hours
329990043	Add '-dcrop' option to ps and svg backends (Closed)	Ebe123	pkx166h, carl.d.sorensen		2 days, 4 hours
326960043	[Enhancement] Add '-dcrop' option to ps and svg backends (Closed)	pkx166h	Ebe123, carl.d.sorensen	8	2 days, 4 hours
327050043	[plaso] Added tests for reading timezone from TZif file #929 (Closed)	Joachim Metz	aaronp		2 days, 5 hours
327080043	Typo fix (Closed)	fujiwara	shawn.p.huang, Peng		3 days, 7 hours
307630043	DCTM: Fix trivial ErrorProne warnings (Closed)	JohnL	Brett Johnson		4 days, 13 hours
322450043	[l2tdevtools] Changes for update to pylint 1.7 #163 (Closed)	Joachim Metz	onager		6 days
327920043	[plaso] Increase test queue timeout to avoid flakes #1253 (Closed)	onager	Joachim Metz		6 days

1 - 20 Older ›