Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 422348

Summary: Refactor the unified diff parser and expose "original file + unified diff" option in the stand alone compare widget.
Product: [ECD] Orion Reporter: libing wang <libingw>
Component: ClientAssignee: libing wang <libingw>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 4.0   
Target Milestone: 5.0 M1   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description libing wang CLA 2013-11-22 10:54:19 EST
There are 3 things we need to do:
1. refactor the unified diff parser, using regEx to parse all the diff hunks.
2. Adding more corner unit test cases
3. Currently we only support two files as input for the stand alone compare widget, which is the most common pattern. But exposing "file + diff" pattern does not hurt anything and may fit some rare cases.
Comment 1 libing wang CLA 2013-11-28 10:04:43 EST
fixed with http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=10e4027281ed95ac9c34db8e37a478b958c094bb.
Realized that the compare widget already exposed the "file + unified diff" option, just need to expose that in the demo.