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 :	save comment
<Esc> :	cancel edit

Recent Open Issues

« Newest ‹ Newer 81 - 100 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
508002	DWARF reader doesn't support length for high_pc	mattdr	jimb	4	12 years, 3 months
497002	minidump_stackwalk: Show argument registers for ARM stack traces	Michael Krebs	jimb, Ted Mielczarek		12 years, 3 months
502003	Remove dead code	rafael.espindola	Mark Mentovai, jimb	1	12 years, 3 months
502002	mac: Explicitly include unistd.h for getpagesize().	thakis	Mark Mentovai	1	12 years, 4 months
490002	Building fails with gcc 4.1 due to unsupported -Werror= switch	davi.arnaut	Mark Mentovai, Ted Mielczarek	4	12 years, 5 months
484002	Check-in stack_frame_symbolizer.[h\|cc] that were missing in r1068.	SiyangXie	ivanpe.google, Ted Mielczarek, Ivan Penkov		12 years, 5 months
479003	Comment out the (unused) variable name in the function definition	rafael.espindola	jimb, jimb		12 years, 6 months
478004	Record declaration names when they have a DW_AT_MIPS_linkage_name	rafael.espindola	jimb	2	12 years, 6 months
475003	Update comment.	rafael.espindola	jimb		12 years, 6 months
479002	Fix warning "comparing signed and unsigned value"	jakerr	Mark Mentovai, qsr		12 years, 6 months
463003	MIPS support for Dump-Syms	deepthi	Ted Mielczarek, jimb	2	12 years, 6 months
381002	Add MIPS support	chrisdearman	Ted Mielczarek, Ben Chan, deepthi	18	12 years, 6 months
426003	Support relative paths in android/run-checks.sh	digit	Ted Mielczarek, Mark Mentovai	2	12 years, 6 months
322001	Fix AddMappingInfo() to properly zero-terminate the name.	Michael Krebs	Ted Mielczarek		12 years, 6 months
436002	remove getcontenxt usage for Android x86 enabling	sql			12 years, 7 months
386001	A hackish patch to make Breakpad upload the log if it's impossible to create a file in the crashed …	glider	Mark Mentovai, jeremy	51	12 years, 8 months
417002	minidump-2-core: Output all memory regions to core file	Michael Krebs	markus		12 years, 8 months
409002	Remove 1 static initializer.	thakis	Mark Mentovai		12 years, 9 months
387003	Bits necessary to send the reports along with the minidumps.	glider	Mark Mentovai		12 years, 11 months
387002	Fix strange cast of enum	Cris Neckar	cpu		12 years, 11 months

« Newest ‹ Newer 81 - 100 Older ›