simon-git: putty (main): Simon Tatham
Commits to Tartarus hosted VCS
tartarus-commits at lists.tartarus.org
Sat Jan 28 15:04:40 GMT 2023
TL;DR:
658ec045 Move Windows definition of CP_UTF8 into windows subdir.
Repository: https://git.tartarus.org/simon/putty.git
On the web: https://git.tartarus.org/?p=simon/putty.git
Branch updated: main
Committer: Simon Tatham <anakin at pobox.com>
Date: 2023-01-28 15:04:40
commit 658ec0457ff955553dd8414ff270fbc637f87d13
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=658ec0457ff955553dd8414ff270fbc637f87d13;hp=343f64c2ca7a7a76321111185c7593a0cf619d55
Author: Simon Tatham <anakin at pobox.com>
Date: Sat Jan 28 14:53:55 2023 +0000
Move Windows definition of CP_UTF8 into windows subdir.
I've only just noticed that the definition of CP_UTF8 as 65001 (the
Windows code page number for UTF-8) is in the main putty.h, under an
ifdef that checks whether the per-platform header file had already
defined it to something else. That's a silly way to do things! Better
that the Windows-specific definition goes _in_ the Windows platform
header, and putty.h contains no fallback. That way, anyone writing a
third separate platform directory will get an error reminding them
that they have to provide the right definition for their platform,
instead of finding out later via a runtime failure.
putty.h | 3 ---
windows/platform.h | 1 +
2 files changed, 1 insertion(+), 3 deletions(-)
More information about the tartarus-commits
mailing list