> I regret not actually addressing the issues with the goto labels in > https://github.com/openjdk/jdk/pull/15996, where initialization of locals in > sspi were jumped over by gotos to a certain label. I changed the > initializations into split declarations and assignments in > https://github.com/openjdk/jdk/pull/15996, but this is simply a hack and does > not address the real issue of gotos jumping over locals. I've as such fixed > the issues with them properly this time, by simply deleting the labels and > duplicating the code where they're used. As mentioned, this unfortunately > does increase duplicate code, but is the cleanest solution I could come up > with for the labels
Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision: - Merge branch 'openjdk:master' into patch-9 - NULL to nullptr in sspi.cpp - Missed labels in sspi.cpp - Actually resolve issues with goto labels in sspi ------------- Changes: - all: https://git.openjdk.org/jdk/pull/16682/files - new: https://git.openjdk.org/jdk/pull/16682/files/1e33e810..f4108192 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=02-03 Stats: 75439 lines in 2183 files changed: 42985 ins; 22345 del; 10109 mod Patch: https://git.openjdk.org/jdk/pull/16682.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/16682/head:pull/16682 PR: https://git.openjdk.org/jdk/pull/16682
