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
321380043	ICU ticket #13254: Parameterizes data driven tests.	nrunge	andy.heninger, markus.icu		5 hours, 25 minutes
323340043	ticket:13285 Adding NumberingSystem constructor methods to DecimalFormatSymbols (J and C).	sffc	markus.icu	3	5 hours, 35 minutes
326950044	ticket:13289 Fixing engineering notation discrepancy with the LDML spec.	sffc	markus.icu	1	5 hours, 45 minutes
328140043	Add usage of OpenType font feature to the documents	trueroad	pkx166h, lemzwerg, tisimst	5	5 hours, 59 minutes
326960043	Enhance the -dpreview method for SVG output	pkx166h			19 hours, 29 minutes
328110043	[dfvfs] Changes for pylint 1.7 upgrade	Joachim Metz	onager		20 hours, 36 minutes
328770043	ticket:13152 Adding 'dataLocale' property so that data can come from a different source than the Sy…	sffc	andy.heninger	3	21 hours, 44 minutes
326950043	Hacks to put random battor data in an ETW trace	bruce dawson			1 day, 1 hour
328170043	NS-3 GSoC 2017 LTE CA handover milestone 2	ilabdsf			1 day, 3 hours
321350043	Update URW++ OTF files	trueroad	lemzwerg		1 day, 7 hours
328160043	ns3.22-lte	zorazeali			1 day, 7 hours
321340043	[l2tdevtools] Changes for pylint 1.7 upgrade #163 and making Unicode strings the default #151	Joachim Metz	onager		1 day, 20 hours
326110043	Patch for BCube and Fat-Tree Topology helpers with netanimation support	Shravya			2 days, 9 hours
328050043	Change \note markup command to get a duration	dak	thomasmorley651		2 days, 16 hours
328040043	[plaso] Changed l2t_csv output module to join MACB timestamps #306	Joachim Metz	onager	43	2 days, 18 hours
328060043	ticket:12519 BreakIterator assignment not handling locales	andy.heninger	markus.icu	8	3 days, 6 hours
321310043	Make window.inner{width,height} use layout viewport	ajuma			3 days, 8 hours
329880043	Announce slur ends when they are known	dak			3 days, 15 hours
317900043	"Bug 2546: Add PrintRoutingTable(All)Change function to only print routing table changes."	ammo6818-vandals.uidaho.edu	Tom Henderson, mariemarth7	8	4 days, 15 hours
322370043	[plaso] Changed to setup.py to exclude non-script files from bdist_rpm #1299	Joachim Metz	aaronp		5 days, 16 hours

1 - 20 Older ›