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 Issues

« Newest ‹ Newer 501 - 520 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
429002	Clean up warnings about narrowing conversion (Closed)	Jess	Ben Chan, mmentovai, Mark Mentovai	4	13 years, 3 months
427003	Prepending with "src/" is undesirabe in #include directives (Closed)	Ivan Penkov	digit, Mark Mentovai, Ted Mielczarek		13 years, 3 months
428002	Compatibility with AddressSanitizer (fast memory error detector for C/C++) (Closed)	Ivan Penkov	ivanpe.google, Ted Mielczarek, Mark Mentovai	8	13 years, 3 months
424002	android: Add --abi option to android/run-checks.sh (Closed)	digit	Ted Mielczarek, Mark Mentovai	3	13 years, 3 months
426002	Remove some linux libc calls from client library (take2) (Closed)	digit	Ted Mielczarek, Mark Mentovai		13 years, 3 months
411002	Remove some linux libc calls from client library. (Closed)	digit	Mark Mentovai, Ted Mielczarek	8	13 years, 3 months
416002	Android ucontext_t handling refinement. (Closed)	digit	Mark Mentovai, Ted Mielczarek, jimb	11	13 years, 3 months
408002	Add the capability to include an arbitrary data stream within minidumps (Closed)	Cris Neckar	cpu, Mark Mentovai, cpu1, ted, Ted Mielczarek, Michael Krebs	26	13 years, 3 months
421002	Linux: Fix a bunch of clang errors from not handling return values. (Closed)	Lei Zhang (chromium)	Mark Mentovai	11	13 years, 3 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, 3 months
415002	Move minidump_extension_linux.h contents into minidump_format.h (Closed)	Ted Mielczarek	Mark Mentovai, Lei Zhang (chromium)	1	13 years, 3 months
417002	minidump-2-core: Output all memory regions to core file	Michael Krebs	markus		13 years, 3 months
418002	Add some more DWARF ARM register numbers (Closed)	Ted Mielczarek	jimb	1	13 years, 3 months
393002	Rework dump_symbols.cc to handle cross-word-size dumping (Closed)	Ted Mielczarek	jimb, jimb, digit, Mark Mentovai	23	13 years, 3 months
405002	Add MD_OS_ANDROID definition. (Closed)	digit	Mark Mentovai, Ted Mielczarek	22	13 years, 3 months
392002	Refactor some ELF bits from file_id.cc into elfutils.{h,cc} (Closed)	Ted Mielczarek	jimb, Mark Mentovai	1	13 years, 3 months
406002	Allow the crash generation server to be initialized with a handle instead of a pipe name. (Closed)	bsmedberg	Ted Mielczarek	10	13 years, 3 months
407002	Add Android NDK module definition + sample application (Closed)	digit	Mark Mentovai, Ted Mielczarek	68	13 years, 3 months
410002	Allow the minidump writer to collect handle data so that resulting dump contains information about … (Closed)	alexeypa	Mark Mentovai, Ted Mielczarek, justin	9	13 years, 3 months
412002	Fix compilation directive for PPC. (Closed)	qsr	Mark Mentovai		13 years, 4 months

« Newest ‹ Newer 501 - 520 Older ›