javaee / javahelp

javahelp
Other
14 stars 15 forks source link

IllegalArgumentException from MergingSearchEngine.merge(MergingSearchEngine.java:91) #22

Open glassfishrobot opened 16 years ago

glassfishrobot commented 16 years ago

Random issue. No single scenarion how to reproduce. We have 15 reports about it. It happens when we call constructor JHelp(HelpSet) ie. when Help window is opened for the first time and JHelp instance is created. Original report is at http://www.netbeans.org/issues/show_bug.cgi?id=127368. List of duplicates is at http://statistics.netbeans.org/analytics/detail.do?id=3169.

Environment

Operating System: All Platform: All

Affected Versions

[current]

glassfishrobot commented 16 years ago

Reported by mslama

glassfishrobot commented 15 years ago

jglick said: Obviously a high priority; 252 duplicates.

Error reporting in jh.jar is abysmal. MergingSearchEngine.engines is apparently permitted to contain null, which it should not; MergingSearchEngine(NavigatorView) ought to throw IAE in case it cannot construct the engine. makeEngine can just return null, which it should not - should throw IAE with a descriptive message in case params == null, or in case !params.containsKey("data"), or in case konstructor.newInstance(args) throws InvocationTargetException (do not use System.err or printStackTrace). Should assume JDK 1.4+ and use exception causes properly - never swallow a throwable and throw a fresh Error!

glassfishrobot commented 16 years ago

Was assigned to javahelp-issues

glassfishrobot commented 7 years ago

This issue was imported from java.net JIRA JAVAHELP-22