[Scummvm-devel] [Scummvm-cvs-logs] SF.net SVN: scummvm:[38376] scummvm/trunk/engines/sci/gfx/gfx_res_options. cpp
Max Horn
max at quendi.de
Mon Feb 16 22:34:24 CET 2009
Am 16.02.2009 um 20:20 schrieb Max Horn:
>
> Am 16.02.2009 um 19:40 schrieb anotherguest at users.sourceforge.net:
>
>> Revision: 38376
>> http://scummvm.svn.sourceforge.net/scummvm/?
>> rev=38376&view=rev
>> Author: anotherguest
>> Date: 2009-02-16 18:40:36 +0000 (Mon, 16 Feb 2009)
>>
>> Log Message:
>> -----------
>> isblank is not available on all platforms. Needs to be handled in a
>> different way.
>
> Where is it not available? Symbian?
>
> We could simply provide
>
> #define scummvm_isblank(c) ((c) == ' ' || (c) == '\t' || (c) == '\n'
> || (c) == '\r')
>
> in either a sci specific header, or if other stuff uses it, in
> scummsys.h.
>
> That would on the long run probably be more portable than trying to
> redefine isblank.
Looking at this again, my def of isblank was of course wrong, it
should only check space and tab (and Lars actually provided code for
that, whoops :-). Anyway, I wonder if isblank could be replaced by
isspace there.
Furthermore, I wonder about ./gfx/gfx_res_options.cpp -- for what is
this code used? It also seems to contain some unused stuff, like
gfx_update_conf() -- in fact, removing that function and then removing
all the stuff that was only used by it, gets rid of all occurances of
isblank(). Now I wonder whether it would be safe to just do that...
Bye,
Max
More information about the Scummvm-devel
mailing list