<div dir="ltr"><div dir="ltr">On Mon, Nov 11, 2024 at 8:58 AM Maurizio Cimadamore <<a href="mailto:maurizio.cimadamore@oracle.com">maurizio.cimadamore@oracle.com</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

  
  <div>
    <blockquote type="cite"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div>Should I create issues for the following? <span style="font-family:arial,sans-serif">Let me know your
                thoughts:</span><br>
            </div>
            <div>
              <ul>
                <li>Minor lint refactoring cleanups<br>
                </li>
                <li>Add a new <span style="font-family:monospace">warn()</span>
                  compiler step after flow analysis</li>
                <ul>
                  <li>ThisEscapeAnalyzer is a charter member<br>
                  </li>
                </ul>
                <li>Refactor to move lint warnings into the warn() step
                  (umbrella)</li>
                <ul>
                  <li>Add issues over time under this umbrella for
                    migrating existing warnings<br>
                  </li>
                  <li>Depends on: Add new warn() step</li>
                </ul>
                <li>Umbrella issue for the <span style="font-family:monospace">@SuppressWarnings </span>cleanup</li>
                <ul>
                  <li>Existing cleanup issues go under here<br>
                  </li>
                </ul>
                <li>Suppression warnings (this proposal)</li>
                <ul>
                  <li>Depends on: Add new warn() step</li>
                  <li>Depends on: Umbrella issue for the <span style="font-family:monospace">@SuppressWarnings </span>cleanup</li>
                  <li>Depends on: Minor lint refactoring cleanups</li>
                </ul>
              </ul>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <p>Yep. Not sure whether this proposal depends on the
      @SuppressWarnings cleanup: in principle you could also add the
      analysis and disable it when building the JDK, and let the various
      components to be updated at their pace.</p></div></blockquote><div>Thanks. Here are the issues I've created...</div><div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Minor lint refactoring cleanups</blockquote><div><br></div><div>JDK-8344079: Minor fixes and cleanups to compiler lint-related code</div></div><div><div class="gmail_quote"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Add a new <span style="font-family:monospace">warn()</span>
                  compiler step after flow analysis</blockquote><div><br></div><div>JDK-8344148: Add an explicit compiler phase for warning generation</div><div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Refactor to move lint warnings into the warn() step
                  (umbrella)</blockquote><div><br></div><div>I figured this one can come later. <br></div><div><br></div></div></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> Umbrella issue for the <span style="font-family:monospace">@SuppressWarnings </span>cleanup</blockquote><div> </div><div>JDK-8344076: Remove unnecessary @SuppressWarnings annotations and Xlint:-foo options (umbrella)</div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Suppression warnings (this proposal)</blockquote><div><br></div>JDK-8344159: Add lint warnings for unnecessary warning suppression<br><div><br></div><div>There are also PR's for the first two.</div><div><br></div><div>Thanks,<br></div><div>-Archie</div><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature">Archie L. Cobbs<br></div></div>