Bug 236909 - Problem with undo on text editor
Summary: Problem with undo on text editor
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 5.0   Edit
Hardware: PC Linux-GTK
: P3 normal with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-12 12:30 EDT by Frédéric Belouin CLA
Modified: 2020-09-04 15:17 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Belouin CLA 2008-06-12 12:30:11 EDT
Build ID: I20080523-0100

Steps To Reproduce:
1. open a new C/C++ file
2. remove a character on the code
3. build the solution
4. try to undo, --> strange eclipse comportment


More information:
I have seen it on eclipse C/C++
And I think the problem also exist in eclipse RCP

I know that Ganymede R3 is not a release, but I'm testing it.

Thank you for your work
Comment 1 Remy Suen CLA 2008-06-12 16:35:31 EDT
(In reply to comment #0)
> 4. try to undo, --> strange eclipse comportment

What do you mean by "strange eclipse comportment"?

> I have seen it on eclipse C/C++
> And I think the problem also exist in eclipse RCP

So you have not seen this problem opening other non-C/C++ files, is this correct?
Comment 2 Frédéric Belouin CLA 2008-06-16 11:44:43 EDT
I try it with eclipse RCP and I am not able to reproduce the bug 
so I think it is only a c/c++ bug

when I undo, there are some characters (empty) that are added to the file.

Regards
Comment 3 Anton Leherbauer CLA 2008-06-17 04:08:07 EDT
(In reply to comment #2)
> when I undo, there are some characters (empty) that are added to the file.

This is expected behavior.
The C/C++ editor removes trailing whitespace on save by default, therefore the first undo step adds the trailing whitespace again.

Unfortunately, each removed whitespace sequence is undone separately. I think we should collapse this into a single compound change.