Re: RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation [v2]

2020-09-16 Thread Joel Borggrén-Franck
On Tue, 15 Sep 2020 15:18:20 GMT, Rafael Winterhalter 
 wrote:

>> This patch assures that an annotation on a type bound is placed on the 
>> correct bound index.
>
> Rafael Winterhalter has refreshed the contents of this pull request, and 
> previous commits have been removed. The
> incremental views will show differences compared to the previous content of 
> the PR.

Marked as reviewed by jfranck (Reviewer).

-

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


Re: RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation

2020-09-15 Thread Rafael Winterhalter
On Tue, 15 Sep 2020 11:31:58 GMT, Joel Borggrén-Franck  
wrote:

>> Pre-git review thread here: 
>> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-June/067049.html
>
> @raphw test is missing imports and doesn't compile, other than that looks 
> good.

Bad copy paste. Fixed on the branch.

-

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


Re: RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation [v2]

2020-09-15 Thread Rafael Winterhalter
> This patch assures that an annotation on a type bound is placed on the 
> correct bound index.

Rafael Winterhalter has refreshed the contents of this pull request, and 
previous commits have been removed. The
incremental views will show differences compared to the previous content of the 
PR. The pull request contains one new
commit since the last revision:

  8202473: A type variable with multiple bounds does not correctly place type 
annotation

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/155/files
  - new: https://git.openjdk.java.net/jdk/pull/155/files/6541a83f..775b3630

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

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

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


Re: RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation

2020-09-15 Thread Joel Borggrén-Franck
On Tue, 15 Sep 2020 08:10:00 GMT, Joel Borggrén-Franck  
wrote:

>> This patch assures that an annotation on a type bound is placed on the 
>> correct bound index.
>
> Pre-git review thread here: 
> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-June/067049.html

@raphw test is missing imports and doesn't compile, other than that looks good.

-

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


Re: RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation

2020-09-15 Thread Joel Borggrén-Franck
On Mon, 14 Sep 2020 19:49:08 GMT, Rafael Winterhalter 
 wrote:

> This patch assures that an annotation on a type bound is placed on the 
> correct bound index.

Pre-git review thread here: 
https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-June/067049.html

-

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


RFR: 8202473: A type variable with multiple bounds does not correctly place type annotation

2020-09-14 Thread Rafael Winterhalter
This patch assures that an annotation on a type bound is placed on the correct 
bound index.

-

Commit messages:
 - 8202473: A type variable with multiple bounds does not correctly place type 
annotation

Changes: https://git.openjdk.java.net/jdk/pull/155/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=155=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8202473
  Stats: 80 lines in 2 files changed: 77 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/155.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/155/head:pull/155

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