July 9, 2017
11:02 a.m.
On 09.07.2017 14:00, Nikolay Sivov wrote: > 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. > Ah, maybe it is, because destination buffer length is never zero.