On 09.07.2017 13:42, Stefan Dösinger wrote: > >> + if (!src || !dst) >> + return 0; > This is redundant. WideCharToMultiByte checks this for you and returns > 0. I noticed this after sending the _To_wide patch and will send a patch > removing the redundant check from _To_wide. > > Checking 'dst' is not redundant though.