yes, the test project did not have the pref set to "error". Note however setting this particular one to warning is goofy since the resultant code does not have a hope of running. it really is an error. We are talking about Forbidden access not Discouraged.
Jeff John Arthorne/Ottawa/[EMAIL PROTECTED] Sent by: [EMAIL PROTECTED] 08/29/2007 06:33 PM Please respond to Equinox development mailing list <[email protected]> To Equinox development mailing list <[email protected]> cc Subject Re: [equinox-dev] restricted access setttings I think Jeff was saying that the template has that preference set to error, not that the template has an error. This deviation from the template for the test project was intentional though, since tests sometimes need to access internals. The tests also intentionally deviate from the conventions for the "unexternalized strings" setting (ignore instead of warning). Perhaps we need a different convention for tests/tools that aren't part of shipping code. Tom, for the deprecation warning, you can avoid that by marking the implementation method as deprecated as well (a deprecated method that implements a deprecated interface method is not flagged as a warning). Thomas Watson <[EMAIL PROTECTED]> Sent by: [EMAIL PROTECTED] 08/29/2007 09:30 AM Please respond to Equinox development mailing list <[email protected]> To Equinox development mailing list <[email protected]> cc Subject Re: [equinox-dev] restricted access setttings Which template has the error? The jdt prefs template at ( http://www.eclipse.org/equinox/documents/org.eclipse.jdt.core.prefs) has the following values for discouraged and forbidden access: org.eclipse.jdt.core.compiler.problem.discouragedReference=error org.eclipse.jdt.core.compiler.problem.forbiddenReference=error On another note I find it inconvenient that we set "Signal overriding or implementing deprecated method" to warning. In some cases we have interfaces which have deprecated methods, there is no choice for the implementer, they must implement the deprecated method. Tom Jeff McAffer ---08/28/2007 11:42:40 PM---I was playing around in the provisioning test bundle and spent about 1 hour tracking down a classloading issues that turned out Jeff McAffer <[EMAIL PROTECTED]> Sent by: [EMAIL PROTECTED] 08/28/2007 11:40 PM Please respond to Equinox development mailing list <[email protected]> To [email protected] cc Subject [equinox-dev] restricted access setttings I was playing around in the provisioning test bundle and spent about 1 hour tracking down a classloading issues that turned out to be a missing package import. This should have shown up as an error but it turns out that the Test project has forbidden access set to Warning instead of Error. Please be sure that all projects conform to the Equinox coding guidelines (the template has it as error). Where there are deviations, let people know. for example, I left the test project set as Warning for discouraged use as tests sometimes need to reach in to access things not normally available. It might be worthwhile somehow scannng the projects to confirm that they are all set appropriately. Perhaps a simple tool that just looks at the relevant settings files? Jeff _______________________________________________ equinox-dev mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/equinox-dev _______________________________________________ equinox-dev mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/equinox-dev _______________________________________________ equinox-dev mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/equinox-dev
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
<<image/gif>>
_______________________________________________ equinox-dev mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/equinox-dev
