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

« Newest ‹ Newer 501 - 520 Older ›