This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 45608 - Null values & NPE in diff settings
Summary: Null values & NPE in diff settings
Status: RESOLVED FIXED
Alias: None
Product: utilities
Classification: Unclassified
Component: Diff (show other bugs)
Version: 4.x
Hardware: PC Linux
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords:
: 45672 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-06-27 03:45 UTC by _ tboudreau
Modified: 2004-08-13 12:10 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

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 _ tboudreau 2004-06-27 03:45:44 UTC 
When I select the Diff Settings node in Options,
all of the properties have "<Lookup null value>"
as their value;  when I click any one, an NPE is
thrown.

The NPE is mine (ObjectEditor...), but something
strange is going on with the values.

FWIW, IMO this settings node should be deleted
anyway - it's a perfect example of settings no
user will ever care about, and it's available in
the diff UI anyway.
Comment 1 Martin Entlicher 2004-07-08 11:38:42 UTC 
For me it works O.K., but I get the NPE as well. But only for the
"Default Diff Service" property.
Provided that there is only one diff service anyway, we can remove
that property.

The other two works O.K. to me. They are useful for setting the
default diff engine/visualizer you want to use. It's true that it can
be selected in the diff GUI, but if you want to use e.g. an external
diff instead of built-in diff always, it's not practical to change it
in the diff GUI each time you invoke it.
You can also uncheck "Show Diff Service Selector" property of
"Built-in Diff Service" in which case you do not get the selector in
the diff GUI to save the screen space.
Comment 2 Martin Entlicher 2004-07-08 11:57:43 UTC 
"Default Diff Service" property removed. The other two work O.K. to me.

/cvs/diff/src/org/netbeans/modules/diff/DiffSettingsBeanInfo.java,v 
<--  DiffSettingsBeanInfo.java
new revision: 1.10; previous revision: 1.9
Comment 3 Martin Entlicher 2004-07-08 12:27:08 UTC 
*** Issue 45672 has been marked as a duplicate of this issue. ***
Comment 4 _ tboudreau 2004-07-12 05:31:01 UTC 
> The other two works O.K. to me. They are useful for setting the
> default diff engine/visualizer you want to use. It's true that it can
> be selected in the diff GUI, but if you want to use e.g. an external
> diff instead of built-in diff always, it's not practical to change it
> in the diff GUI each time you invoke it.

So have the diff system remember the last selected engine/visualizer, and you can still 
solve the problem and get rid of the setting too.
Comment 5 Martin Entlicher 2004-07-12 14:55:06 UTC 
So, changing the engine/visualizer in the diff GUI will change the
defaults. This can be done easily. This behavior can be handy anyway.
I've submitted issue #46041 for that.