Status: New
Owner: ----
Labels: Type-Enhancement Priority-Medium

New issue 2654 by ryan.bea...@gmail.com: Set, jump between bookmarks in DiffViewer
http://code.google.com/p/reviewboard/issues/detail?id=2654

What version are you running?
1.6.1

What's the URL of the page this enhancement relates to, if any?
???

https://reviewboard.example.com/r/12345/diff/#index_header

Describe the enhancement and the motivation for it.

Developer posts a sizable review with a fair number of functions spread across multiple files. During review, I want to sanity check calls into new function whose return value, memory allocations, etc. I don't know. So, I need to jump to the called function, see what it does, what its side effects are, etc., and then return to the call to see if things check out. (For example, did the called function return a pointer to something allocated from the heap, and does the current function need to free it in case of failure?)

With the current diff viewer, I have to manually search for the definition of the new function, then manually go back to its call site. It'd be nice if I could
   1.  record my current location in the diff viewer,
   2.  seek somewhere else in the diff, and then
3. press a key/keys or click something to immediately jump back to the saved location.

What operating system are you using? What browser?
Google Chrome 18–19 on Linux and Windows 7.

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.

Reply via email to