Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323512 - Builder options not getting copied
Summary: Builder options not getting copied
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build-managed (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 8.0   Edit
Assignee: Doug Schaefer CLA
QA Contact: Chris Recoskie CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-24 11:36 EDT by Doug Schaefer CLA
Modified: 2010-08-24 12:23 EDT (History)
0 users

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 Doug Schaefer CLA 2010-08-24 11:36:22 EDT 
There is a missing call to copyChildren in the Builder "copy" constructor. Also, I need to remove the isDirty from the builder and reuse the one in it's parent class.
Comment 1 Doug Schaefer CLA 2010-08-24 11:41:54 EDT 
Code checked in.
Comment 2 CDT Genie CLA 2010-08-24 12:23:03 EDT 
*** cdt cvs genie on behalf of dschaefer ***
Bug 323512 - Removed local isDirty to inherit from HoldsOption. Called super copyChildren in the copy constructor.

[*] Builder.java 1.49 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/Builder.java?root=Tools_Project&r1=1.48&r2=1.49