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
321180043	Implemented client labels. (Closed)	scudette			3 days, 1 hour
321170043	ticket:13151 Removing obsolete assertion from PluralFormat.java (Closed)	sffc	andy.heninger		3 days, 1 hour
325880043	ticket:13129 Changing default Parse MathContext to 34 digits instead of 16 digits. (Closed)	sffc	andy.heninger	2	3 days, 9 hours
323950043	Implement flow download via a shell script. (Closed)	scudette			4 days, 2 hours
321150044	[plaso] Moved Travis tests to edge group test machine images (Closed)	Joachim Metz	romaing		4 days, 14 hours
327830043	[dfdatetime] Moved Travis tests to edge group test machine images (Closed)	Joachim Metz	onager		4 days, 15 hours
325890043	[dfwinreg] Moved Travis tests to edge group test machine images (Closed)	Joachim Metz	onager		4 days, 15 hours
321160043	[dfvfs] Moved Travis tests to edge group test machine images (Closed)	Joachim Metz	onager		4 days, 15 hours
322120043	openerp2tryton: Update to 4.4 (Closed)	ced			4 days, 18 hours
322050043	[l2tdevtools] Add beautifulsoup build (for dftimewolf). (Closed)	onager	Joachim Metz	2	4 days, 20 hours
316580043	[l2tdevtools] Update regex to parse ZLib version in HTML (Closed)	romaing	Joachim Metz, jberggren		4 days, 23 hours
325850043	[l2tdevtools] Add a prebuild configuration option for projects (Closed)	romaing	onager		5 days
327780043	[plaso] Make end-to-end tests Windows friendly (Closed)	romaing	onager	7	5 days
322030043	[l2tdevtools] Several fixes for missing project version (Closed)	Joachim Metz	jberggren		5 days, 3 hours
317630043	[plaso] Refactored psort front-end to tool #160 (Closed)	Joachim Metz	jberggren	2	5 days, 4 hours
324950043	ticket:13231 Changing DecimalFormat min/max setters to use the most recent setting instead of alway… (Closed)	sffc	andy.heninger	2	5 days, 7 hours
317710043	ticket:13228 Adding more symbols to standard/localized pattern mapping function (Closed)	sffc	andy.heninger		5 days, 8 hours
323160043	Added CSRF protection to API endpoints. (Closed)	scudette			5 days, 10 hours
312640043	ticket:13177 Moving CompactType to public CompactDecimalFormat.java (Closed)	sffc	andy.heninger	2	5 days, 11 hours
322130043	Implement the Firebase notified. (Closed)	scudette			6 days, 4 hours

1 - 20 Older ›