Bug 45029 - State integrity problem when restoring default values on the 'C/C++ Build' window of the project properties dialog
Summary: State integrity problem when restoring default values on the 'C/C++ Build' wi...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 1.2   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 3.0   Edit
Assignee: Sean Evoy CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-10-16 13:41 EDT by James Ciesielski CLA
Modified: 2005-01-28 15:04 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description James Ciesielski CLA 2003-10-16 13:41:02 EDT 
Using the 'Restore Defaults' button on the 'C/C++ Build' window of the project 
properties dialog without clicking on 'OK' or 'Apply' creates problems.

Test case:
1. Create a managed C/C++ executable project.
2. Add a file, call it 'main.cpp' and insert the main() routine.
3. Right-click on the project and select its properties.
4. Select 'C/C++ Build' from the list box on the left.
5. Under the 'C/C++ Compiler tree, select the 'Warnings' child.
6. Manipulate the checkboxes so that they are selected in a NON-default manner.
7. Click 'Apply'.
8. Click 'Restore Defaults'.
9. Click 'Cancel'.

At this point, one would expect that the defaults would not have been 
persisted because the user clicked 'Cancel' and NOT 'Apply' or 'OK'.

10. Re-open the project properties and review the 'Warning' checkboxes.  Note 
that the default values are displayed.

In case it helps:
11. Click 'Cancel'.
12. Close the project.
13. Re-open the project.
14. Re-open the project properties and review the 'Warning' checkboxes.  Note 
that the values are no longer the default values.


NB: This is a problem with all of the settings under 'C/C++ Build', not just 
with the contents of the 'Warning' child.
Comment 1 Alain Magloire CLA 2003-10-16 19:47:59 EDT 
Reassign to the managed build owner.
Comment 2 Kleo Hapitas CLA 2004-07-07 16:34:22 EDT 
PR was targeted to the 2.0 release but not resolved, moving target to 2.1
Comment 3 Sean Evoy CLA 2004-11-01 16:51:58 EST 
Definitely still a problem. Milestone set for 3.0
Comment 4 Leo Treggiari CLA 2004-12-08 16:45:45 EST 
In CDT 2.1, I added a "confirmation" dialog box when the user selects Reset 
Defaults.  It says:

------------------------------------------------------------------------

This action will reset all of the tools in the selected configuration to their 
default settings.

Do you want to proceed?

------------------------------------------------------------------------

The defaults are then set and saved - Cancel will not undo the Reset Defaults 
button.

I'm not sure what you mean by steps 11 - 14.

------------------------------------------------------------------------

In case it helps:
11. Click 'Cancel'.
12. Close the project.
13. Re-open the project.
14. Re-open the project properties and review the 'Warning' checkboxes.  Note 
that the values are no longer the default values.

------------------------------------------------------------------------

In CDT 2.1, if I continue on with these steps after step 10, in step 14, I see 
the default values - i.e. Restore Defaults reset the defaults and applied the 
changes.
Comment 5 Sean Evoy CLA 2005-01-28 15:04:19 EST 
Sounds like this is fixed in 2.1.