JDK-8267187: Remove deprecated constructor for Log #4037
Conversation
|
👋 Welcome back jjg! A progress list of the required criteria for merging this PR into |
|
@jonathan-gibbons The following labels will be automatically applied to this pull request:
When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing lists. If you would like to change these labels, use the /label pull request command. |
openjdk
bot
added
javadoc
|
@jonathan-gibbons This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be: You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 3 new commits pushed to the
Please see this link for an up-to-date comparison between the source branch of this pull request and the ➡️ To integrate this PR with the above commit message to the |
Reviewed-by: mcimadamore
…onospaced font Reviewed-by: erikj, serb
… causes crash Reviewed-by: iklam, minqi
Reviewed-by: vlivanov, coleenp
Co-authored-by: Jatin Bhateja <jbhateja@openjdk.org> Reviewed-by: neliasso, kvn
Reviewed-by: prappo, iris, vromero
…chFileException Reviewed-by: asemenyuk, herrick
…s no spec Reviewed-by: aivanov
…e by keyboard Reviewed-by: aivanov, dmarkov
Reviewed-by: sgehwolf, egahlin
Reviewed-by: thartmann, redestad
Reviewed-by: mcimadamore
Co-authored-by: Marcus G K Williams <mgkwill@openjdk.org> Co-authored-by: Stefan Johansson <sjohanss@openjdk.org> Co-authored-by: Thomas Stuefe <stuefe@openjdk.org> Reviewed-by: stuefe, sjohanss
Reviewed-by: psandoz, kvn
…nostic 8256301: ObjectMonitor::is_busy() should return bool Reviewed-by: coleenp, pchilanomate
Reviewed-by: serb
Reviewed-by: mgronlun
Reviewed-by: kvn, thartmann
Reviewed-by: azeemj, vlivanov, jiefu
…a failed with OutOfMemoryError Co-authored-by: xiangyuan <xiangyuan@tencent.com> Reviewed-by: stuefe
…LibraryLookup is gone Reviewed-by: zgu, jiefu
…ashed with OutOfMemory Reviewed-by: tschatzl, kbarrett
…267904 Reviewed-by: kvn, dlong
…view) Co-authored-by: Brian Goetz <briangoetz@openjdk.org> Co-authored-by: Mandy Chung <mchung@openjdk.org> Co-authored-by: Jan Lahoda <jlahoda@openjdk.org> Reviewed-by: mcimadamore, forax, godin, psandoz, mchung
Reviewed-by: ayang, tschatzl, kbarrett
Reviewed-by: stefank, dholmes
Reviewed-by: jjg
Reviewed-by: vlivanov, kvn
…art of the decorator set specified at creation. Reviewed-by: dholmes, phh
Reviewed-by: stefank
…Spec Reviewed-by: prappo, darcy
Reviewed-by: ayang, sjohanss
Reviewed-by: zgu
Reviewed-by: jlahoda
… runs Reviewed-by: lancea, jjg, erikj
…_node() Reviewed-by: rkennke, roland
Reviewed-by: sjohanss
…in JTreg compiler tests Co-authored-by: Christian Hagedorn <chagedorn@openjdk.org> Co-authored-by: Tobias Hartmann <thartmann@openjdk.org> Reviewed-by: iignatyev
Reviewed-by: iris, erikj, jjg
Reviewed-by: prr
… streams Reviewed-by: naoto, alanb
Reviewed-by: erikj
|
|
openjdk
bot
added
ready
|
PR seems confused. New PR filed as #4404 |
In the course of other work, I came across this deprecated constructor in Log which just existed for a public entry point in javadoc which has already been removed.
There is one remaining use in javadoc, which can be changed to use an alternate non-deprecated constructor.
Progress
Issue
Reviewers
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk pull/4037/head:pull/4037$ git checkout pull/4037Update a local copy of the PR:
$ git checkout pull/4037$ git pull https://git.openjdk.java.net/jdk pull/4037/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 4037View PR using the GUI difftool:
$ git pr show -t 4037Using diff file
Download this PR as a diff file:
https://git.openjdk.java.net/jdk/pull/4037.diff