On Mon, 22 Dec 2025 20:48:36 GMT, Damon Nguyen <[email protected]> wrote:
>> This issue is not currently causing any problems, but I am adding specifiers >> to explicitly print these strings. This is to avoid any issues down the line >> where changes to the lines building the string can cause any accidental >> formatting. Overall, this should be a harmless update and improves the >> stability and reliability of printing these strings. > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revision: > > Update printf src/jdk.accessibility/windows/native/jabswitch/jabswitch.cpp line 231: > 229: fprintf(origFile, > 230: > "assistive_technologies=com.sun.java.accessibility.AccessBridge\n" > 231: "screen_magnifier_present=true\n"); I don't think it is what Alexander meant. I think he meant it should look like fprintf(origfile, "%s", "assistive_technologies=com.sun.java.accessibility.AccessBridge\n"screen_magnifier_present=true\n"); I am not sure doing the above would even resolve the complaint. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28949#discussion_r2641321297
