Re: RFR: 8256585: Remove in-place conversion vector operators from Vector API [v2]

2020-11-19 Thread Paul Sandoz 
On Thu, 19 Nov 2020 19:47:15 GMT, Sandhya Viswanathan 
 wrote:

>> Remove partially implemented in-place conversion vector operators from 
>> Vector API:
>>ofNarrowing, ofWidening, INPLACE_XXX
>
> Sandhya Viswanathan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Update documentation

Marked as reviewed by psandoz (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/1305

Re: RFR: 8256585: Remove in-place conversion vector operators from Vector API [v2]

2020-11-19 Thread Sandhya Viswanathan 
On Thu, 19 Nov 2020 17:18:12 GMT, Paul Sandoz  wrote:

>> Sandhya Viswanathan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Update documentation
>
> The documentation `Vector.convert` and `Vector.convertShape` needs to be 
> updated to remove specification of in-place conversions, as does the class 
> documentation on `Vector`. A search for the term `in-place` should find the 
> relevant locations.

Updated documentation to reflect removal of widening and contracting in-place 
conversions.

-

PR: https://git.openjdk.java.net/jdk/pull/1305

Re: RFR: 8256585: Remove in-place conversion vector operators from Vector API [v2]

2020-11-19 Thread Sandhya Viswanathan 
> Remove partially implemented in-place conversion vector operators from Vector 
> API:
>ofNarrowing, ofWidening, INPLACE_XXX

Sandhya Viswanathan has updated the pull request incrementally with one 
additional commit since the last revision:

  Update documentation

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1305/files
  - new: https://git.openjdk.java.net/jdk/pull/1305/files/73122deb..db9d37de

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=1305=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=1305=00-01

  Stats: 12 lines in 1 file changed: 0 ins; 12 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1305.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1305/head:pull/1305

PR: https://git.openjdk.java.net/jdk/pull/1305

Re: RFR: 8256585: Remove in-place conversion vector operators from Vector API

2020-11-19 Thread Paul Sandoz 
On Thu, 19 Nov 2020 03:26:20 GMT, Sandhya Viswanathan 
 wrote:

> Remove partially implemented in-place conversion vector operators from Vector 
> API:
>ofNarrowing, ofWidening, INPLACE_XXX

The documentation `Vector.convert` and `Vector.convertShape` needs to be 
updated to remove specification of in-place conversions, as does the class 
documentation on `Vector`. A search for the term `in-place` should find the 
relevant locations.

-

Changes requested by psandoz (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1305

RFR: 8256585: Remove in-place conversion vector operators from Vector API

2020-11-18 Thread Sandhya Viswanathan 
Remove partially implemented in-place conversion vector operators from Vector 
API:
   ofNarrowing, ofWidening, INPLACE_XXX

-

Commit messages:
 - 8256585: Remove in-place conversion vector operators from Vector API

Changes: https://git.openjdk.java.net/jdk/pull/1305/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=1305=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8256585
  Stats: 121 lines in 1 file changed: 0 ins; 118 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1305.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1305/head:pull/1305

PR: https://git.openjdk.java.net/jdk/pull/1305