On Tue 16.Mar'10 at 17:39:33 +0100, Tamas TEVESZ wrote:
> On Tue, 16 Mar 2010, Carlos R. Mafra wrote:
>
> > > but as wstrdup() has already bothered me
> > > while I was writing wmgenmenu for not having a const qualifier,
> >
> > Right, my memory really sucks. It was wtokensplit() actually,
> > for which I thought the first argument could be constant.
> > But I never really thought more than 5 seconds on that,
> > which may also explain my dim memory about it :-)
>
> :)
>
> do shove this in please while there, let's keep this tidy, now that
> we've gone over it.
Yep, I folded into your "Kill redundant function" patch and the result
is already in #next.
>
> --- a/src/dialog.c
> +++ b/src/dialog.c
> @@ -182,9 +182,9 @@ static char *HistoryFileName(char *name)
> return filename;
> }
>
> -static int matchString(void *str1, void *str2)
> +static int strmatch(const void *str1, const void *str2)
> {
> - return (strcmp((char *)str1, (char *)str2) == 0 ? 1 : 0);
> + return !strcmp((const char *)str1, (const char *)str2);
> }
>
> static WMArray *LoadHistory(char *filename, int max)
>
>
> --
--
To unsubscribe, send mail to [email protected].
