Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 313783 - External settings extension point should log a warning if the ext setting provider can't be found
Summary: External settings extension point should log a warning if the ext setting pro...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 7.0   Edit
Hardware: PC Linux-GTK
: P3 normal (vote)
Target Milestone: 7.0   Edit
Assignee: James Blackburn CLA
QA Contact: Doug Schaefer CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-20 13:25 EDT by James Blackburn CLA
Modified: 2010-07-28 15:29 EDT (History)
0 users

See Also:

Attachments
trivial fix (2.86 KB, patch)
2010-05-20 13:25 EDT, James Blackburn CLA 		jamesblackburn+eclipse: iplog-
Details | Diff
View All 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 2010-05-20 13:25:13 EDT 
Created attachment 169381 [details]
trivial fix

I tripped over this while working on bug 310819.

If you specify an external setting provider id, and you get the ID wrong (all too easy to do as the extension point ID is auto-prefixed with the plugin ID). CDT silently ignores the failed ext setting provider.  This is bad and likely one reason extenders struggle to use this extension point.

Trivial patch to log an error if the external provider couldn't be instantiated once per session.
Comment 1 James Blackburn CLA 2010-05-20 13:27:04 EDT 
Fixed.