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

« Newest ‹ Newer 81 - 100 Older ›