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 Open Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
308490043	Doc: update toplevel-score-handler definition reference	pwm			7 hours, 15 minutes
310260043	[eccemotus] Migrating files to public repo. Migrating parsers.	vlejd	Joachim Metz, adamsh, jberggren, romaing, onager		1 day, 6 hours
305530043	[plaso] Changes for running tests with tox #511	Joachim Metz	onager		1 day, 7 hours
308970043	[plaso] Made tests conditional for sdist size	Joachim Metz	onager	14	1 day, 7 hours
309960043	[plaso] Updated review script.	Joachim Metz	jberggren, romaing, onager		1 day, 8 hours
303250043	[plaso] Added attribute container identifiers #771	Joachim Metz	onager	6	1 day, 8 hours
305120043	[plaso] Code clean up of knowledge base #241	Joachim Metz	onager	4	1 day, 9 hours
306940043	[plaso] Removed GetPathAttributes from knowledge base #241	Joachim Metz	onager	14	1 day, 12 hours
306070043	[plaso] Refactored preprocess plugins to directly use knowledge base #241	Joachim Metz	onager	44	1 day, 12 hours
304440043	[plaso] Worked on migrating plaso timestamp to dfdatetime #910	Joachim Metz	onager		1 day, 13 hours
310970043	Remove chronicle ID from getUpdateAcls	Srinivas	JohnL	2	1 day, 21 hours
308460043	Greek Uppercasing	andy.heninger	andy.heninger	4	1 day, 23 hours
303100043	Java SpoofChecker	sffc	andy.heninger	93	2 days
303660043	DCTM: Use a traverser for modified ACLs	JohnL	Brett Johnson, Srinivas	1	2 days, 1 hour
309540043	TBR: Revert "i#1569 AArch64: Reimplement ELF import/export iterators."	zhaoqin	edmund.grimley.evans		2 days, 1 hour
311760043	i#1680 large pages: Replace VMM_BLOCK_SIZE with an option.	Edmund.Grimley.Evans	bruening		2 days, 6 hours
302700043	ui/gtk3: support scroll event in candidates panel	Peng Wu	shawn.p.huang, fujiwara, maiku.fabian	4	2 days, 9 hours
310850043	i#1569 AArch64: Fix drx_buf on AArch64.	Edmund.Grimley.Evans	zhaoqin	9	2 days, 12 hours
308030043	Initial commit of Rekall agent.	scudette			2 days, 14 hours
309530043	i#2006 generalize drcachesim: add analysis_tool_t	zhaoqin	bruening		2 days, 20 hours

1 - 20 Older ›