> 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 incrementally with one additional commit since the last revision: std:: qualifier sspi.cpp ------------- Changes: - all: https://git.openjdk.org/jdk/pull/16682/files - new: https://git.openjdk.org/jdk/pull/16682/files/f7bb7fb0..c0084f23 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=08 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16682&range=07-08 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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