Bug 234850 - None of Binary Parser Options Text Fields of AbstractGNUBinaryParserPage are persisted on Apply / OK
Summary: None of Binary Parser Options Text Fields of AbstractGNUBinaryParserPage are ...
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 4.0.3   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:
: 445235 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-05-30 09:31 EDT by James Blackburn CLA
Modified: 2020-09-04 15:23 EDT (History)
5 users (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 Blackburn CLA 2008-05-30 09:31:16 EDT 
Build ID: 4.0.3

Steps To Reproduce:
In Project Properties > C/C++ Build > Settings > Binary parsers
none of the "Binary Parser Options" fields are persisted on Apply / OK.


More information:
It looks like BinaryParsTab.java should perform the apply via calling informPages(true) but this never occurs. The result is that AbstractGNUBinaryParserPage.java:performApply() never appears to be called.

Looking at the code, this still seems to be a problem in HEAD
Comment 1 Francois Rigault CLA 2009-05-19 11:42:10 EDT 
This is still not fixed in CDT 5.0.2.200902130801
Comment 2 Andrew Gvozdev CLA 2009-06-28 15:52:44 EDT 
(In reply to comment #0)
> Looking at the code, this still seems to be a problem in HEAD
Why, the feature appears to be documented in JavaDoc:
	/* Settings from binary parser pages are NOT saved by prior CDT version.
	 * So existing binary parsers _always_ use default values.
	 * Moreover, obsolete interface is used while attempting to save. 
	 * 
	 * We have to affect both parsers and pages 
	 * to teach them to save data really.
	 * 
	 * It will be done in next versions. Currently pages are disabled.
	 */
Some plans are outlined there... Too bad the planmakers are not around anymore.
Comment 3 Corey Ashford CLA 2011-04-06 22:04:31 EDT 
Still seeing this in CDT 7.0.2.
Comment 4 Otavio Pontes CLA 2012-01-09 06:55:09 EST 
I am facing this problem in CDT 8.0.0 too.
Comment 5 Andrew Gvozdev CLA 2014-10-01 17:30:59 EDT 
*** Bug 445235 has been marked as a duplicate of this bug. ***