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
4173068	HTTP and SOCKS proxy support for ApacheHttpTranport	vially	belovic.martin, jinny7845, yanivi		6 days, 14 hours
183680043	Issue 501 - Improve the logging, provide TraceSource logger	peleyal	molegend69xz, 周		1 month
6493067	Test code review post	ItzmE29	vazquezfajardo.juanalberto810, gallion342, baby546833, itzme29, bichphuong02121993, mylinh442000, shambrika42, catsteve4200, Tonya, 周		1 month
566080043	Clean up and fix glyph contour generation	hanwenn	Dan Eble, hahnjo, lemzwerg, Noe Renteria, aaireill111	10	1 month, 1 week
1758041	code review 1758041: io/ioutil: add CopyFile	Kyle Lemons	eliarconcepcion, adg, 周	23	1 month, 2 weeks
4515165	XPC Demo: missing 'base.js' ref in relay.html	carsten.kraus			1 month, 3 weeks
4571064	Fix rounded corner issue FF4 and upgrade IE9 to css rounded corners	Johans			1 month, 3 weeks
560030044	Remove deprecated context properties	Valentin Villenave	thomasmorley651, roypjohnson76, lemzwerg	1	1 month, 4 weeks
6452058	code review 6452058: goauth2: adding a jwt package to support server to serv...	gavaletz_google	gavaletz, iant2, adg, adg1, jbd, jcgregorio_google, 周, shaan7, gun20toro, bradfitz	28	2 months
6501079	Allow lru_cache to accept any type for which equality is defined	ghais	vitess, sougou		3 months
545890043	flask-tryton: Add support for 5.6 series	pokoli	ced	1	4 months, 3 weeks
277230044	class of service	Tom Henderson	Stefano Avallone, Tommaso Pecorella	10	6 months, 1 week
555820043	flask-tryton: Convert records ids to string before joining them	pokoli	ced		7 months, 1 week
554030043	Issue 4182: avoid checking the offset of cross-staff stems too early	barrykp	Dan Eble, lemzwerg	6	10 months
569700043	Split glyph contours in up/down segments for skylines	hanwenn	dak, lemzwerg, hahnjo	17	10 months
576090043	Fix #5964: MM Rests shouldn’t segfault when there’s no StaffSymbol.	Valentin Villenave	Dan Eble, dan, carl.d.sorensen	6	10 months
576110043	break substitution cleanup	hanwenn			10 months
569740046	Simplify and speed up break substitution	hanwenn			10 months
572130043	Break substitution cleanup	hanwenn			10 months
567580043	Stop smobifying Transform	hanwenn	dak		10 months

1 - 20 Older ›