Bug 143239 - [options] Deprecate DISABLED value for CORE_ENABLE_CLASSPATH_EXCLUSION_PATTERNS and CORE_ENABLE_CLASSPATH_MULTIPLE_OUTPUT_LOCATIONS
Summary: [options] Deprecate DISABLED value for CORE_ENABLE_CLASSPATH_EXCLUSION_PATTER...
Status: VERIFIED WONTFIX
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.2   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: 3.5 M3   Edit
Assignee: Philipe Mulet CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-23 11:40 EDT by Maxime Daniel CLA
Modified: 2008-10-28 13:05 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maxime Daniel CLA 2006-05-23 11:40:00 EDT
3.2 RC5
Those Java core options use values ENABLED and DISABLED. In contrast with compiler options, which in case of a DISABLED value tolerate an unchanged input and simply do not exhibit a supplementary function associated to the said option (see COMPILER_PB_SUPPRESS_WARNINGS for example), the two options toggle in fact between the two folliwing logical states:
- the function is enabled;
- the function is not enable *and* the other parameters associated to the said function must be removed.
I believe that an ENABLE / FORBIDDEN values pair would underline the real behavior and clarify the contrast with the general 'don't use extraneous parameters when disabled' policy. Of course, this would need a gradual deprecation of DISABLED in this case.

(Alternatively, the option to simply ignore secondary parameters might be considered. I assume though that the design was elaborated with a precise end in mind, and that the check on the absence of such parameters is needed by some clients of the API.)
Comment 1 Olivier Thomann CLA 2006-10-10 21:59:26 EDT
This would be a breaking API change.
Philippe, do we still want to do it or simply improve the doc to reflect Maxime's statement?
Comment 2 Philipe Mulet CLA 2008-09-17 07:28:30 EDT
No action planned. We will leave with the current story.
Comment 3 Kent Johnson CLA 2008-10-28 13:05:08 EDT
Verified for 3.5M3 using I20081026-2000 build.