On Wed, 11 Jan 2023 21:16:51 GMT, Mikael Vidstedt <mik...@openjdk.org> wrote:
> This PR does not address all the remaining sprintf:s in hotspot, and with it > now explicitly forbidden the build will fail: > This is a question to me as well. I noticed there are still some use of sprintf, but the building passed on MacOS and Linux. I was wondering if the following update really work (if the '...' parameter works for the forbidden?), or something else matters. FORBID_C_FUNCTION(int sprintf(char*, const char*, ...), "use os::snprintf"); > I count ~30 sprintf:s that need updating. > > I'm also curious: some of the sprintfs are C2 (src/hotspot/share/opto) - are > your builds including C2? If so, why are you not running into the issue for > those files? I'm new to hotspot. Do you know how could I enable C2? Thanks! ------------- PR: https://git.openjdk.org/jdk/pull/11935
