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
326260043	[plaso] Changes to dependency checks	Joachim Metz	onager		7 minutes
324260043	Implementation of warm-ups + sync'd start times	w.fakher.reda	w.fakher.reda, kirill.sc	23	28 minutes
323450043	Doc: Issue 4993 Different time signatures in alternative repeats	Trevor Daniels	dak	1	2 hours, 40 minutes
327150043	[plaso] Changed tests files to include multiple tags #1037	Joachim Metz	jberggren		2 hours, 48 minutes
323460043	[l2tdocs] Added documentation about the l2tbinaries packaging and releasing process #8	Joachim Metz	onager		4 hours, 3 minutes
323440043	Add system-by-system output to SVG backend	Ebe123			13 hours, 38 minutes
328310043	ticket:13327 Fixing buffer overflow in NumberingSystem constructor.	sffc	markus.icu		1 day, 11 hours
328290043	[plaso] Terminate failing processes more forcefully	onager	Joachim Metz, aaronp		1 day, 20 hours
325270043	[plaso] Changes to ignore byte-order mark and clean up single line pyparsing text parsers	Joachim Metz	onager	1	2 days, 4 hours
328280043	[dfvfs] Added text encoding support to text file helper	Joachim Metz	onager		2 days, 4 hours
322490043	[plaso] Added Linux system product preprocessor plugin #1337	Joachim Metz	aaronp	4	2 days, 7 hours
321460043	Issue 1434: Fix text spacing in SVG	Ebe123	pkx, dak		2 days, 9 hours
330040043	Doc: Add how to change the notation fonts	trueroad	tisimst	6	2 days, 9 hours
322500043	ICU ticket #13253: Parameterizes data driven test, replaces encoded characters in test data with Un…	nrunge	andy.heninger, markus.icu	2	2 days, 15 hours
330010043	ICU ticket #13236: revert some changes from r40307 so that, 1., TestIDNA passes when run in Eclipse…	nrunge	andy.heninger, mscherer.icu	1	2 days, 16 hours
307380043	CSD CodeReview for Group 3 : v5.0 see details in README file	lihongzhouchn	shokhin, nirjhor.kuet, armen, dkostic, theodoor	37	3 days, 6 hours
307370043	TCP measurement relative scripts	pandan0315	sunnygu0928, njwade1233, khalidomer88, dkostic, Xiaodong Zhang, yiangmuyu	26	3 days, 6 hours
312040043	This is the file of guest VM startup configurations.	Xiaodong Zhang	sunnygu0928, njwade1233, khalidomer88, pandan0315, dkostic, yiangmuyu	3	3 days, 6 hours
312990043	xen_three_flows	pandan0315	sunnygu0928, njwade1233, khalidomer88, dkostic, Xiaodong Zhang, yiangmuyu	3	3 days, 6 hours
318790043	add rtt cdf plot	pandan0315	sunnygu0928, njwade1233, khalidomer88, dkostic, Xiaodong Zhang, yiangmuyu		3 days, 6 hours

1 - 20 Older ›