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
330640043	adding test.py (Closed)	younes.tahri3	engr.sikandar.iqbal, 2016shana2333, Vasileios Papageorgiou, dkostic, paul.busi38, michail.xirouchakis, alireza.farshin		12 hours, 23 minutes
329470043	[plaso] Made Unicode strings the default in Windows Registry parser plugins tests #1268 (Closed)	Joachim Metz	aaronp	2	1 day
327670043	[plaso] Code clean up of tools (Closed)	Joachim Metz	aaronp, onager	11	1 day, 9 hours
332040043	[dfvfs] Textual fixes of docstrings and comments (Closed)	Joachim Metz	onager		1 day, 10 hours
331020043	[l2tdevtools] Added configuration option to control dpkg source format (Closed)	Joachim Metz	onager		1 day, 23 hours
335810043	[l2tpreg] Made Unicode strings the default and moved PregTool to separate file (Closed)	Joachim Metz	onager		1 day, 23 hours
336730043	[l2tpreg] Added dpkg files #13 and updated dependencies (Closed)	Joachim Metz	onager		1 day, 23 hours
336740043	[l2tpreg] Merged preg front-end and tool (Closed)	Joachim Metz	onager		1 day, 23 hours
332900043	[l2tpreg] Updated setup.py with more elegant bdist_msi version work-around #14 (Closed)	Joachim Metz	onager		1 day, 23 hours
330570043	[plaso] Made Unicode strings the default in bencode, cookie, ESEDB, OLECF, plist and syslog parser … (Closed)	Joachim Metz	aaronp	2	1 day, 23 hours
331980043	[plaso] Made Unicode strings the default in parsers tests #1268 (Closed)	Joachim Metz	aaronp		1 day, 23 hours
336840043	ui/gtk3: Increase timeout of window filter in PropertyPanel (Closed)	fujiwara	shawn.p.huang, Peng	2	2 days, 1 hour
331750043	[plaso] Fix MacOS install script (Closed)	onager	Joachim Metz, aaronp	1	2 days, 5 hours
332890043	[plaso] Replace countable usages of storage with store (Closed)	onager	Joachim Metz	1	2 days, 5 hours
332850043	[l2tdevtools] Add review script functionality to l2tdevtools (Closed)	onager	Joachim Metz	12	2 days, 6 hours
335870043	[dfvfs] Changes for the backports.lzma dependency #229 (Closed)	Joachim Metz	onager		2 days, 9 hours
336810043	[plaso] Changed pysqlite dpkg name to match distribution (Closed)	Joachim Metz	romaing		2 days, 20 hours
329510043	ui/gtk3: Unref Panel and Indicator when double panels run (Closed)	fujiwara	shawn.p.huang, Peng	1	3 days, 2 hours
334850043	[dfvfs] Changed pysqlite dpkg name to match distribution (Closed)	Joachim Metz	onager		4 days
330590043	[l2tpreg] Changed pysqlite dpkg name to match distribution (Closed)	Joachim Metz	onager		4 days

1 - 20 Older ›