Re: [PATCH] [WinHelp]: fix for 5314

[Dmitry Timoshkov]

From: "Eric Pouech"  wrote:
> +#include <ctype.h>

...
> -
>            if (!ch || !isalpha(ch)) break;

Since that's apparently a Windows encoding shouldn't this be
GetStringType() instead of isalpha()?

[PATCH] [WinHelp]: fix for 5314

[Eric Pouech]

Dmitry Timoshkov a écrit :
> From: "Eric Pouech"  wrote:
> > +#include <ctype.h>
>
> ...
> > -
> >            if (!ch || !isalpha(ch)) break;
>
> Since that's apparently a Windows encoding shouldn't this be
> GetStringType() instead of isalpha()?

no, the issue (here) is about creeping control characters in the string 
(which shows a bogus file, not a mishandling of encoding)
and also, it appears that the string is 7bit ascii, so isalpha is sufficient
A+