The JDWP spec was suppose to be updated for all addInstanceFilter() APIs with 
this minor clarification:


     /**
      * Restricts the events generated by this request to those in
      * which the currently executing instance is the object
    - * specified.
    + * specified as determined by applying the "==" operator.
      * <P>
      * Not all targets support this operation.
      * Use {@link VirtualMachine#canUseInstanceFilters()}


BreakpointRequest was used as the template and copied to the javadoc of the 
other event request's addInstanceFilter(). However, BreakpointRequest had a 
pre-existing omission, and as a result this omission got applied to all the 
other addInstanceFilter() javadocs:


     /**
      * Restricts the events generated by this request to those in
    - * which the currently executing instance ("this") is the object
    - * specified.
    + * which the currently executing instance is the object
    + * specified as determined by applying the "==" operator.
      * <P>
      * Not all targets support this operation.
      * Use {@link VirtualMachine#canUseInstanceFilters()}


The reference to "this" was incorrectly removed since it was missing from 
BreakpointRequest. This PR adds "this" back in, and also fixes 
BreakpointRequest to also include the "this" reference. 

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

-------------

Commit messages:
 - Add 'this' reference back into addInstanceFilter() javadoc

Changes: https://git.openjdk.org/valhalla/pull/2514/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2514&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8385965
  Stats: 10 lines in 10 files changed: 0 ins; 0 del; 10 mod
  Patch: https://git.openjdk.org/valhalla/pull/2514.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2514/head:pull/2514

PR: https://git.openjdk.org/valhalla/pull/2514

Reply via email to