> this test case is being rejected by javac:
> 
> class Test {
>     Test(Test t) {}
> 
>     Test(String s) {
>         this(new Test());
>     }
> 
>     Test() {}
> }
> 
> here what is happening is that when analyzing the `new Test()` invocation the 
> code to determine if there is an enclosing instance for this expression, 
> javac is not checking first if there is an enclosing instance at all. This 
> analysis shouldn't be done for cases like the one above for which there is no 
> enclosing instance.
> 
> TIA

Vicente Romero has updated the pull request incrementally with one additional 
commit since the last revision:

  another iteration

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2231/files
  - new: https://git.openjdk.org/valhalla/pull/2231/files/4ae16a93..3049d4d8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2231&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2231&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/valhalla/pull/2231.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2231/head:pull/2231

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

Reply via email to