#2459 Enable WBEMListener get/setProperty

New_Feature
closed-fixed
Dave Heller
5
2012-09-14
2012-05-23
Dave Blaschke
No
0 up votes | 0 down votes | 0%
11 comments

Currently, WBEMListener getProperty() just returns null while setProperty() just throws an IllegalArgumentException. After WBEMListenerFactory starts returning unique instances of WBEMListener(SBLIM) (see bug 3529062), these APIs need to function as expected.

Discussion

1 2 > >> (Page 1 of 2)
  • Dave Blaschke
    Dave Blaschke
    2012-05-23

    Proposed WBEMListenerTest.java

     
    Attachments
  • Dave Blaschke
    Dave Blaschke
    2012-05-23

    • assigned_to: nobody --> hellerda
     
  • Dave Blaschke
    Dave Blaschke
    2012-05-23

    • labels: --> Java Client (JSR48)
    • milestone: --> New_Feature
     
  • Dave Heller
    Dave Heller
    2012-06-02

    Patch sent for community review. During a 2 week period any
    exploiter may comment on the patch, request changes or turn it
    down completely (with good reason). For the time being the patch is part
    of the "Experimental" branch in CVS.

     
  • Dave Blaschke
    Dave Blaschke
    2012-06-03

    • status: open --> open-fixed
     
  • Dave Blaschke
    Dave Blaschke
    2012-06-20

    This patch introduced three javac warnings that should be fixed before promoting to the HEAD branch:

    1> Read access to enclosing field WBEMListenerSBLIM.iConfiguration is emulated by a synthetic accessor method. Increasing its visibility will improve your performance WBEMListenerSBLIM.java jsr48-client-error/src/org/sblim/cimclient line 105
    2> The import java.io.File is never used WBEMListenerSBLIM.java jsr48-client-error/src/org/sblim/cimclient line 34
    3> Unqualified access to the field WBEMListenerSBLIM.iConfiguration WBEMListenerSBLIM.java jsr48-client-error/src/org/sblim/cimclient line 105

     
  • Dave Heller
    Dave Heller
    2012-06-20

    The below mentioned compiler warnings have been resolved. Committed to CVS Head

     
  • Dave Blaschke
    Dave Blaschke
    2012-06-21

    • status: open-fixed --> pending-fixed
     
  • Dave Blaschke
    Dave Blaschke
    2012-06-21

    The community review has completed and we received no substantial criticism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

     
1 2 > >> (Page 1 of 2)