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
194066	code review 194066: Add an explicit patent grant to the Go license. (Closed)	iant	rsc, goprojectowner		1 day, 7 hours
5756065	code review 5756065: test: actually run them on windows (Closed)	brainman	rsc		6 days, 15 hours
2473041	code review 2473041: misc: add goplay (Closed)	adg	r, rsc	42	6 days, 15 hours
1740043	code review 1740043: math: amd64 version of log (Closed)	Charlie Dorian	rsc		6 days, 15 hours
2289041	code review 2289041: gc, spec, tests: no auto-indirect of pointer to interfa... (Closed)	rsc	r, gri	4	6 days, 15 hours
1711048	code review 1711048: freetype/raster: make cap and join functions instead of... (Closed)	nigeltao	adg	2	6 days, 15 hours
353790043	Use a stable sort when ordering MIDI items (Closed)	bobo1239	Dan Eble, dan, carl.d.sorensen, c_sorensen		1 month
367870043	Issue 5428: Avoid type-conversion warnings related to font glyph index (Closed)	Dan Eble			1 month, 2 weeks
343430043	Consolidate overlapping gsutil text about cp over OS-specific files (Closed)	Mike Schwartz	houglum, nstock		1 month, 2 weeks
325190043	mods per nstock suggestion (Closed)	Mike Schwartz			1 month, 2 weeks
341480043	Changes per review comments (Closed)	Mike Schwartz			1 month, 2 weeks
357800043	Changes per review comments (Closed)	Mike Schwartz			1 month, 2 weeks
349010043	Add test for chunked transfer encoding apitools fix (Closed)	Mike Schwartz	houglum	10	1 month, 2 weeks
353780043	Add documentation advising against using gsutil rsync or cp over directories containing symlinks (Closed)	Mike Schwartz	houglum, nstock	8	1 month, 2 weeks
349790043	Disable multiprocessing (threading-only) under Alpine Linux (Closed)	Mike Schwartz	houglum	13	1 month, 2 weeks
369880043	Add warning about surrounding gsutil args with # in them with quotes (Closed)	Mike Schwartz	yarbrough, houglum, nstock	2	1 month, 2 weeks
363800043	Issue 5426: Use vsize for counts in page-breaking classes (Closed)	Dan Eble			1 month, 3 weeks
206770044	Doc: issue 1432 (Closed)	barrykp	fedelogy, pkx166h, dak, c_sorensen	2	2 months
349780043	Issue 5421: Store page numbers as int type (Closed)	Dan Eble			2 months
348990043	Issue 5415: Fix type-conversion warnings in parser and lexer (Closed)	Dan Eble	dak	5	2 months, 2 weeks

1 - 20 Older ›