vs2019 の導入はまだ先だと思ってました。
これはこれでやってもいいと思います。

vc++の場合特に、ヘッダーには #pragma once が付くので
多めにインクルードしてしまったとしても影響ないと思うからです。
必要なヘッダーをインクルードしてなかった、という問題なので、対処すべきと思います。

インクルードするファイルは、
locale でいいように思いますが、ダメなんですかね？

インクルードするファイルは、
locale でいいように思いますが、ダメなんですかね？

#include <clocale>

がいいという事でしょうか？どちらでも良いと思います。

Visual Studio 2017 で /showIncludes で確認したところ

StdAfx.h
 string
  istream
   ostream
    ios
     xlocnum
      streambuf
       xiosbase
        xlocale
         xlocinfo
          xlocinfo.h
           locale.h

というように locale.h が取り込まれてました。

標準ライブラリのパスが
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023
以下になってますが、新しい開発環境のバージョンでは標準ライブラリの実装が変わって問題が出たという事のようです。

MinGWでビルドした場合でもどこかで勝手に #include していて問題が起きてなかったぽいですね。そちらはどこで #include されていたのかを調べる気は無いです。。

インクルードするファイルは、
locale でいいように思いますが、ダメなんですかね？

#include <clocale>

がいいという事でしょうか？どちらでも良いと思います。

いいえ、です。
locale.h は C 言語向けのランタイムヘッダです。
clocale は c++ でロケール関数を使うためのランタイムヘッダです。

ぼくが言ってるのは locale という c++向けの標準ライブラリヘッダに変えなくていいかってことです。

#include <locale>

厳格な仕様では clocale は std 名前空間の中に locale.h を展開するっぽいです。
locale.h が必要になったんじゃなくて、vs2019が仕様準拠度を高めたために
グローバル ::setlocale を認識しなくなったということなのかも知れんです。
ちゃんと検証してませんが、 setlocale とか std::locale なら通るのかも、とも思うわけです。

・・・なんか外れたこと書いてる気がします。
たぶん、こういう変更があったんじゃないかと思ってます。

詳しいことは c++er の人にお任せするとして、ざっくりとそんな印象を受けてます。

いいえ、です。
locale.h は C 言語向けのランタイムヘッダです。
clocale は c++ でロケール関数を使うためのランタイムヘッダです。

ぼくが言ってるのは locale という c++向けの標準ライブラリヘッダに変えなくていいかってことです。

#include <locale>

厳格な仕様では clocale は std 名前空間の中に locale.h を展開するっぽいです。
locale.h が必要になったんじゃなくて、vs2019が仕様準拠度を高めたために
グローバル ::setlocale を認識しなくなったということなのかも知れんです。
ちゃんと検証してませんが、 setlocale とか std::locale なら通るのかも、とも思うわけです。

https://en.cppreference.com/w/cpp/header/locale
こちらのヘッダですか。

VS2019でビルドすると以下のようなエラーメッセージが出ます。

Severity	Code	Description	Project	File	Line	Suppression State
Error	C2065	'LC_CTYPE': undeclared identifier	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	870	
Error	C3861	'_create_locale': identifier not found	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	870	
Error	C3861	'_free_locale': identifier not found	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	875	
Error	C2065	'LC_CTYPE': undeclared identifier	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	891	
Error	C3861	'_create_locale': identifier not found	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	891	
Error	C3861	'_free_locale': identifier not found	sakura	D:\projects\sakura\sakura_core\util\string_ex.cpp	895	
Error	C2065	'LC_ALL': undeclared identifier	sakura	D:\projects\sakura\sakura_core\_main\WinMain.cpp	67	
Error	C3861	'setlocale': identifier not found	sakura	D:\projects\sakura\sakura_core\_main\WinMain.cpp	67	

_create_locale についてぐぐった結果以下のページが見つかりました。そこには下記の記述が有ったので、#include <locale.h> しました。

https://docs.microsoft.com/ja-jp/cpp/c-runtime-library/reference/create-locale-wcreate-locale?view=vs-2017#requirements

なお、WinMain.cpp ファイル内で使っている setlocale についても必要条件は同じです。

https://docs.microsoft.com/ja-jp/cpp/c-runtime-library/reference/setlocale-wsetlocale?view=vs-2017#requirements

なお、::setlocale という記述で検索しましたがそのような記述は見当たりませんでした。

#include <locale>

して、

std::locale

という記述に変えないと頃す！と言われたらすぐに変えるのでお申し付けください。

・・・なんか外れたこと書いてる気がします。
たぶん、こういう変更があったんじゃないかと思ってます。

No vs ver インクルード方向 説明
1 vs2017以前 clocale ⇒ locale.h locale.h の内容を std 名前空間に展開する
2 vs2019 locale.h ⇒ clocale clocale(std名前空間) を using する
詳しいことは c++er の人にお任せするとして、ざっくりとそんな印象を受けてます。

VS2017環境については berryzplus さんも使っていると思うので、プロジェクトのプロパティの C/C++ の Advanced の Show Includes オプションを Yes (/showIncludes) に変えてビルドしたら Output ウィンドウに #include しているのが全部出るのでそれで実際に確認されると良いと思います。自分はしました。

ああ、ちなみに std::locale はtypoです。
うちのIE11からだとコメント編集できないんで放置してました。
（編集するときはわざわざChrome起ち上げてますｗ）

どうしてこうなった？

beru さんの /showIncludes にヒントがあるのではないでしょうか。

1. サクラエディタのコードでは明示的に #include <locale.h> していなかったが
2. インクルードしているある標準ヘッダが内部的に #include <locale.h> していたので問題にならなかった。
3. VS2019 ではその標準ヘッダが内部的に #include <locale.h> しなくなったので問題が出た。

何も確かめておらず、たぶんですけど。

(追記) 最初の最初に beru さんがもう書いていましたね。>「今まで無くても何で大丈夫だったのかは良く分かりません。どこかの標準ヘッダーで include してるんでしょうか。。」

Review ありがとうございます。Merge します。MinGWでどうして大丈夫だったのかの確認はMSFTがもっと嫌いになったら行おうと思います。

(追記) 最初の最初に beru さんがもう書いていましたね。>「今まで無くても何で大丈夫だったのかは良く分かりません。どこかの標準ヘッダーで include してるんでしょうか。。」

Issue作成前に原因調査を済ませてIssueの本文に明解な記述をしておかないと、レビューワーもトラブルシューティングの習慣で色々な想像をしてしまうという事ですね。お手数おかけしました。。

vs2019では iosbase がxlocaleをインクルードしてくれなくなったってことなんですかね。
std::wistream とか ANSI -> UNICODE 変換が必要なクラスを使うには自前で locale をインクルードせにゃならんのかな・・・。

vs2019では iosbase がxlocaleをインクルードしてくれなくなったってことなんですかね。

VS2019で /showIncludes オプション付けてビルドして出力を確認してみましたが、VS2017の場合と違って、string が istream を include する事が無くなっていました。

string
 xstring
  iosfwd
   yvals.h
    yvals_core.h
     crtdefs.h
     xkeycheck.h

というようになっていて。なお標準ライブラリのパスは
C:\Program Files (x86)\Microsoft Visual Studio\2019\Preview\VC\Tools\MSVC\14.20.27027\
でした。

std::wistream とか ANSI -> UNICODE 変換が必要なクラスを使うには自前で locale をインクルードせにゃならんのかな・・・。

一般論でしか回答出来ませんが、何かの標準ヘッダファイルを include したら違うヘッダファイルも include されるかどうかは処理系依存かもしれないので（規格で決まっているものもあるかもしれませんがいちいち規格参照しながらプログラミングする人はあまりいないと思います）使用する関数やクラスのレファレンスを見て include が必要なヘッダを調べて明示的に書くようにするのが正しいやり方かと思います。面倒なのでよくさぼりがちですけど。

vs2019 preview1は結局いれてないので空想でものを言ってます。
何もなければ sp1 が出るまで入れないはず・・・今のノリだとそのうち入れる気がしますが。

一応、必要なヘッダはできるだけ書く派です。