[GEM-dev] making [pix_multiimage] using glob

Hans-Christoph Steiner hans at at.or.at
Fri Jul 9 18:02:55 CEST 2010


On Jun 21, 2010, at 7:29 AM, IOhannes m zmoelnig wrote:

> On 2010-06-21 09:29, IOhannes m zmoelnig wrote:
>> On 2010-06-18 19:37, Hans-Christoph Steiner wrote:
>>>
>>> Claudius just pointed out that the wildcard * in pix_multiimage is  
>>> not a
>>> wildcard at all, but a %d printf pattern.
>>>
>>> Anyone mind if I replace that code with a real glob?  I have the  
>>> code
>>> for all platforms in the folder_list.c external.
>>>
>>
>> Gem has already abstracted away the code for globbing in Base/ 
>> GemFiles.cpp.
>>
>> however, i would changing the "*" to a real wildcard would totally  
>> break
>> compatibility.
>> i would rather have the asterisk be replaced by something more
>> convenient, e.g. "%d" (and allowing "%06d")
>>
>
> furthermore, i would happily dump the entire [pix_multiimage]  
> altogether
> (and leave it as it is, for legacy reasons), since all it's
> functionality could be replaced with [pix_buffer].
>
> i thought about adding the globbing code to [pix_buffer], but it is
> probably more flexible and less effort, to just use it as it is and  
> use
> [folder_list] or something else to do the globbing.
>
> fgnasdr
> IOhannes

For me, its just really bad form using a * for a printf replacement.   
If you want to make it use printf statements, that's fine too.

.hc

----------------------------------------------------------------------------

News is what people want to keep hidden and everything else is  
publicity.          - Bill Moyers





More information about the GEM-dev mailing list