On Wed, 17 Dec 2025 13:33:25 GMT, Jan Lahoda <[email protected]> wrote:

> Fixing `test/langtools/tools/javac/diags/CheckExamples.java` 
> (`test/langtools/tools/javac/diags/examples/PrimitivePatternMatching.java`) 
> and `test/langtools/tools/javac/preview/PreviewAutoSuppress.java` tests.
> 
> Unfortunately, it is quite hard to make the tests work both with and without 
> `-DDISABLE_PREVIEW_PATCHING=true` (could be done for 
> `PreviewAutoSuppress.java`, but it is very hard to do sensible for 
> `CheckExamples.java`), so given the future is 
> `-DDISABLE_PREVIEW_PATCHING=true`, this change forces this option. (The 
> practical impact of testing only this variant for these two tests is 
> basically zero, I think. We have a different test that verifies all the 
> combinations.)

This pull request has now been integrated.

Changeset: 3c41c2aa
Author:    Jan Lahoda <[email protected]>
Committer: David Simms <[email protected]>
URL:       
https://git.openjdk.org/valhalla/commit/3c41c2aa442076827cb4480373a1d481e481cdf1
Stats:     10 lines in 5 files changed: 3 ins; 5 del; 2 mod

8373826: [lworld] Test PreviewAutoSuppress fails since jimage change
8373828: [lworld] Test CheckExamples fails since jimage change

Reviewed-by: rriggs

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

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

Reply via email to