Re: RFR: 8316971: Add Lint warning for restricted method calls [v4]

2023-10-03 Thread Maurizio Cimadamore
> This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` annotation with a corresponding internal flag. This is > done both

Re: RFR: 8316971: Add Lint warning for restricted method calls [v3]

2023-09-29 Thread Vicente Romero
On Fri, 29 Sep 2023 08:30:07 GMT, Maurizio Cimadamore wrote: >> This patch adds a new lint warning category, namely `-Xlint:restricted` to >> enable warnings on restricted method calls. >> >> The patch is relatively straightforward: javac marks methods that are marked >> with the `@Restricted

Re: RFR: 8316971: Add Lint warning for restricted method calls [v2]

2023-09-29 Thread Vicente Romero
On Fri, 29 Sep 2023 08:14:29 GMT, Maurizio Cimadamore wrote: > > question shouldn't the new Restricted annotation be annotated with the > > @PreviewFeature annotation? it depends on a preview feature > > The Restricted annotation is an internal annotation only. So there is no > value in annot

Re: RFR: 8316971: Add Lint warning for restricted method calls [v3]

2023-09-29 Thread Maurizio Cimadamore
On Fri, 29 Sep 2023 08:22:47 GMT, Maurizio Cimadamore wrote: >> This patch adds a new lint warning category, namely `-Xlint:restricted` to >> enable warnings on restricted method calls. >> >> The patch is relatively straightforward: javac marks methods that are marked >> with the `@Restricted

Re: RFR: 8316971: Add Lint warning for restricted method calls [v3]

2023-09-29 Thread Maurizio Cimadamore
> This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` annotation with a corresponding internal flag. This is > done both

Re: RFR: 8316971: Add Lint warning for restricted method calls [v2]

2023-09-29 Thread Maurizio Cimadamore
On Thu, 28 Sep 2023 15:56:36 GMT, Vicente Romero wrote: > question shouldn't the new Restricted annotation be annotated with the > @PreviewFeature annotation? it depends on a preview feature The Restricted annotation is an internal annotation only. So there is no value in annotating it with `@

Re: RFR: 8316971: Add Lint warning for restricted method calls [v2]

2023-09-28 Thread Vicente Romero
On Thu, 28 Sep 2023 15:36:30 GMT, Maurizio Cimadamore wrote: >> This patch adds a new lint warning category, namely `-Xlint:restricted` to >> enable warnings on restricted method calls. >> >> The patch is relatively straightforward: javac marks methods that are marked >> with the `@Restricted

Re: RFR: 8316971: Add Lint warning for restricted method calls [v2]

2023-09-28 Thread Vicente Romero
On Thu, 28 Sep 2023 15:36:30 GMT, Maurizio Cimadamore wrote: >> This patch adds a new lint warning category, namely `-Xlint:restricted` to >> enable warnings on restricted method calls. >> >> The patch is relatively straightforward: javac marks methods that are marked >> with the `@Restricted

Re: RFR: 8316971: Add Lint warning for restricted method calls [v2]

2023-09-28 Thread Maurizio Cimadamore
> This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` annotation with a corresponding internal flag. This is > done both

Re: RFR: 8316971: Add Lint warning for restricted method calls

2023-09-28 Thread Magnus Ihse Bursie
On Thu, 28 Sep 2023 13:13:31 GMT, Maurizio Cimadamore wrote: > This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` ann

Re: RFR: 8316971: Add Lint warning for restricted method calls

2023-09-28 Thread Maurizio Cimadamore
On Thu, 28 Sep 2023 13:13:31 GMT, Maurizio Cimadamore wrote: > This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` ann

Re: RFR: 8316971: Add Lint warning for restricted method calls

2023-09-28 Thread Maurizio Cimadamore
On Thu, 28 Sep 2023 13:13:31 GMT, Maurizio Cimadamore wrote: > This patch adds a new lint warning category, namely `-Xlint:restricted` to > enable warnings on restricted method calls. > > The patch is relatively straightforward: javac marks methods that are marked > with the `@Restricted` ann

RFR: 8316971: Add Lint warning for restricted method calls

2023-09-28 Thread Maurizio Cimadamore
This patch adds a new lint warning category, namely `-Xlint:restricted` to enable warnings on restricted method calls. The patch is relatively straightforward: javac marks methods that are marked with the `@Restricted` annotation with a corresponding internal flag. This is done both in `Annotat