Github user ham1 commented on a diff in the pull request:

    https://github.com/apache/jmeter/pull/364#discussion_r158429468
  
    --- Diff: src/core/org/apache/jmeter/gui/util/MenuInfo.java ---
    @@ -74,4 +75,16 @@ public String getClassName(){
         public int getSortOrder() {
             return sortOrder;
         }
    +
    +    /**
    +     * Returns whether the menu item represented by this MenuInfo object 
should be enabled
    +     * @param actionCommand    the action command name for the menu item
    +     * @return true when menu item should be enabled, false otherwise.
    +     */
    +    public boolean getEnabled(String actionCommand) {
    +        if (ActionNames.ADD.equals(actionCommand))
    --- End diff --
    
    Please use braces. Checkstyle should pick this up.


---

Reply via email to