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
206770044	Doc: issue 1432 (Closed)	barrykp	fedelogy, pkx166h, dak, c_sorensen	2	1 week
349780043	Issue 5421: Store page numbers as int type (Closed)	Dan Eble			1 week, 2 days
348990043	Issue 5415: Fix type-conversion warnings in parser and lexer (Closed)	Dan Eble	dak	5	3 weeks, 1 day
342260043	Add documentation advising against using gsutil rsync or cp over directories containing symlinks (Closed)	Mike Schwartz	houglum, nstock	4	3 weeks, 2 days
346930044	Issue 5410: Fix warnings in flower (Closed)	Dan Eble			1 month
346950043	goocalendar: Call next week on next page on week view (Closed)	pokoli			1 month
347950043	Issue 5409: Fix many type-conversion warnings in simple-spacer (Closed)	Dan Eble			1 month, 1 week
217560043	Added path to BotsService. (Closed)	cmassaro	Nakaret1992, vadimsh, M-A Ruel, M-A		1 month, 2 weeks
342240043	Issue 5402: Arrange for System::get_bound to return a Paper_column (Closed)	Dan Eble			1 month, 2 weeks
357770043	Issue 5400: Compile with -Wno-cast-function-type when possible (Closed)	Dan Eble	dak	1	1 month, 2 weeks
343370043	Web: fix a few links and typos (issue 5392) (Closed)	fedelogy			1 month, 2 weeks
314790043	Phase 2: ECN implementation for TCP (Closed)	Shravya	Tom Henderson, n.p	22	1 month, 3 weeks
351760043	Issue 5401: Suppress -Wimplicit-fallthrough in g++ >= 7 (Closed)	Dan Eble			1 month, 3 weeks
346900043	Issue 5399: Reuse find_context_above() for ly:context-find (Closed)	Dan Eble			2 months
369820043	Removes text that indicates windows support. (Closed)	alanmorales	tamasz		2 months
343350043	Addressing First Round Of Comments (Closed)	jamaalbwells			2 months, 1 week
363730043	Enhanced perfdiag command to record if running on GCE, GCE instance details, bucket location and st… (Closed)	Mike Schwartz	houglum		2 months, 2 weeks
305380043	Convert a bunch of C++ internals to degrees rather than radians (Closed)	dak	thomasmorley651, carl.d.sorensen, Carl	2	2 months, 2 weeks
306710043	Avoid a few trivial lambda function wrappers (Closed)	dak	simon.albrecht	8	2 months, 2 weeks
318010043	Correct the guns for hire list under "Community" (Closed)	dak			2 months, 2 weeks

1 - 20 Older ›