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
8134043	code review 8134043: go/build: disable cgo when cross compiling (Closed)	dfc	jankostraka.js, bradfitz	2	3 weeks, 5 days
573730043	Use binary search in Axis_group_interface::combine_pure_heights. (Closed)	hanwenn	lemzwerg, hahnjo	1	1 month, 1 week
4907041	code review 4907041: builtin: add documentation for builtins (Closed)	r	gri, dsymonds, rsc, niemeyer	23	1 month, 1 week
842043	PEP 3147 implementation (Closed)	barry	Benjamin, pjenvey, Antoine Pitrou, brett.cannon, Georg, gl	172	2 months
348040043	Web: update Productions list (Closed)	fedelogy	ababialex9, lilypond-pkx, fundasecgin32	15	3 months, 2 weeks
112320044	Don't calculate deltas when viewing issues because that's done in a task (Closed)	jrobbins (chromium)	jrobbins (corp), ajaramillo1629, iannucci, lehmanpstrik, mcourtney1961		3 months, 3 weeks
5476044	upload.py -m --message option refactoring (Closed)	techtonik	Andi Albrecht, mr.gibson40, GvR, M-A	5	6 months, 3 weeks
6310062	URLFetcher has moved from content to net, update references. (Closed)	thakis	Afsar, akalin	2	8 months, 1 week
545910043	Documentation: Remove deprecated skeleton-update (Closed)	hahnjo	fedelogy, vazquezfajardo.juanalberto810		9 months, 4 weeks
5307080	code review 5307080: builtin: document built-in error type (Closed)	adg	rsc, r, r2, illia1991nabatov, bradfitz, rog	3	1 year
2025041	code review 2025041: build: update, streamline documentation for new $GOBIN (Closed)	rsc	ambernopalduvall, r, adg, 幹你娘	7	1 year
561790043	Remove define causing a warning (Closed)	hahnjo	Dan Eble, lemzwerg		1 year
548030043	Add option to use Ghostscript API instead of forking (Closed)	hahnjo	Valentin Villenave, dak, lemzwerg, hanwenn	7	1 year
567610043	flask-tryton: Set _request in default context (Closed)	ced			1 year, 1 month
557680044	Remove reimplementation of linear_interpolate() from stencil-integral (Closed)	hanwenn			1 year, 6 months
551690046	Shortcut Rational addition if either operand is zero (Closed)	hanwenn	Dan Eble, lemzwerg, hahnjo	5	1 year, 6 months
559870043	Fix comment for Box::unite (Closed)	hanwenn	Dan Eble, lemzwerg	2	1 year, 6 months
557780044	output-distance: print summary as final step (Closed)	hanwenn			1 year, 6 months
577820043	Build slur skyline out of segments rather than boxes. (Closed)	hanwenn	hahnjo	2	1 year, 6 months
581960043	Thread skyline construction through stencil interpretation (Closed)	hanwenn	dak, hahnjo	15	1 year, 6 months

1 - 20 Older ›