Changeset 4974 in josm for trunk/src/org


Ignore:
Timestamp:
2012-02-18T00:45:30+01:00 (12 years ago)
Author:
Don-vip
Message:

Preferences tabs: robustness to null icons + code cleanup

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java

    r4970 r4974  
    319319            }
    320320        }
    321 /*
    322         for (Iterator<PreferenceSetting> it = settings.iterator(); it.hasNext();) {
    323             try {
    324                 PreferenceSetting ps = it.next();
    325                 long start = System.currentTimeMillis();
    326                 ps.addGui(this);
    327                 System.out.println(ps.getClass()+" -> "+(System.currentTimeMillis()-start));
    328             } catch (SecurityException e) {
    329                 it.remove();
    330             } catch (Throwable e) {
    331                 // allow to change most settings even if e.g. a plugin fails
    332                 BugReportExceptionHandler.handleException(e);
    333             }
    334         }*/
    335321        addGUITabs(false);
    336322    }
    … …  
    365351                if (expert || !tps.isExpert()) {
    366352                    // Get icon
    367                     ImageIcon icon = ImageProvider.get("preferences", tps.getIconName());
     353                    String iconName = tps.getIconName();
     354                    ImageIcon icon = iconName != null && iconName.length() > 0 ? ImageProvider.get("preferences", iconName) : null;
    368355                    if (settingsInitialized.contains(tps)) {
    369356                        // If it has been initialized, add corresponding tab(s)
    … …  
    376363            }
    377364        }
    378         /*for (PreferenceTab tab : tabs) {
    379             TabPreferenceSetting s = tab.getTabPreferenceSetting();
    380             if (expert || !s.isExpert()) {
    381                 addTab(null, ImageProvider.get("preferences", s.getIconName()), tab.getComponent(), "<html>"+s.getTooltip()+"</html>");
    382             }
    383         }*/
    384365        try {
    385366            if (sel != null) {
Note: See TracChangeset for help on using the changeset viewer.