You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@jmeter.apache.org by bu...@apache.org on 2016/03/17 02:46:18 UTC

[Bug 59185] New: SavePropertyDialog.setSaveConfig/getSaveConfig not used; should be removed

https://bz.apache.org/bugzilla/show_bug.cgi?id=59185

            Bug ID: 59185
           Summary: SavePropertyDialog.setSaveConfig/getSaveConfig not
                    used; should be removed
           Product: JMeter
           Version: 2.13
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: sebb@apache.org

The SavePropertyDialog class is currently only used by AbstractVisualizer.

It seems unlikely it will be needed anywhere else, and anyway the get/set
methods for the saveConfig field don't really make sense. The config is passed
in via the constructor.

If the setter were dropped, the field could be made final.

Not much point having a getter either as the field is provided by the caller.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59185] SavePropertyDialog.setSaveConfig/getSaveConfig not used; should be removed

Posted by bu...@apache.org.

https://bz.apache.org/bugzilla/show_bug.cgi?id=59185

Sebb <se...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

-- 
You are receiving this mail because:
You are the assignee for the bug.