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

1 - 20 Older ›