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
320710043	[dfvfs] Changes to replace stat with attributes #52	Joachim Metz	onager		1 hour, 22 minutes
320700043	ui/gtk3: Integrate custom rendering to use HarfBuzz glyph info	fujiwara	shawn.p.huang		2 hours, 43 minutes
327850043	NS-3 GSoC 2017 LTE CA handover milestone 1	ilabdsf			6 hours, 4 minutes
322730043	trytond-gis: Initial commit	nicoe	ced	33	12 hours, 41 minutes
323180043	[dfvfs] Changes to replace stat with attributes #52	Joachim Metz	onager		14 hours, 15 minutes
323820043	[dfvfs] Fix bug with negative seeks for encrypted, encoded, and compressed streams. #183	dc3.plaso	Joachim Metz, bluenlonesome83		21 hours, 36 minutes
326740043	[plaso] Refactored options to CLI helpers #160	Joachim Metz	aaronp	15	1 day, 15 hours
328780043	ICU ticket #13239: moves to its own file a currently disabled test that has to run independently.	nrunge	mark, markus.icu		1 day, 18 hours
327840043	[plaso] Add Windows end2end test script, and remove unused ones.	romaing	Joachim Metz	2	2 days, 1 hour
317660043	WIP: Implement elementsFromPoint	ajuma			2 days, 15 hours
316670043	[plaso] Refactored options to CLI helpers #160	Joachim Metz	romaing	2	2 days, 18 hours
312700043	[dfvfs] Add support for zip files with missing directory entries. #194	dc3.plaso	onager	6	2 days, 19 hours
328770043	ticket:13152 Adding 'dataLocale' property so that data can come from a different source than the Sy…	sffc	andy.heninger	1	2 days, 23 hours
320680043	ticket:12690 Adding convenience overloads for java.util.Currency.	sffc	andy.heninger		3 days, 10 hours
324960043	ICU ticket #13240: fixes a code coverage test for hashCode method in DateFormat class.	nrunge	andy.heninger, markus.icu		3 days, 14 hours
328750043	Compiler path	alexk_es	Manus, jfiat_es	1	3 days, 15 hours
320690043	Update Finnendahl testimony info	git			3 days, 16 hours
324930043	[dftimewolf] This PR contains a big revamp of dftimeflow's code.	tomchop1	someguyiknow, onager	4	3 days, 18 hours
322140043	DCTCP Patch	Shravya			4 days, 15 hours
321150043	ICU ticket #13242: changes locale to da in an uppercase-come-first collation ICU4C test and re-enab…	nrunge	andy.heninger, markus.icu	2	5 days, 2 hours

1 - 20 Older ›