On 08/07/2012 07:27 PM, John Stoffel wrote:
>
> vinit> The current version of geeqie1.1 that vladimir has putted, is
> vinit> not been able to recognize the image files if the file names of
> vinit> these file do not end with one of the standard image file
> vinit> extension. I think the application should look into the
> vinit> directory for all the image files based on their format rather
> vinit> than their file extension. like how the nautilus, the default
> vinit> file browser for gnome does. it does it for all media files. i
> vinit> think geeqie should have this feature.
>
> It's a good idea, but maybe have it as an option instead?  If the
> option is on (default off), geeqie does the equivilent of a 'file'
> command on each file to see if it can determine the image type.
>
> And if you want to whip up a patch, even just an example one, I'm sure
> that would be an incentive to get it into the app.

If it's an option, it should default to off.  Having to fork() and 
read() for every single file in a directory is drastically more 
expensive than just doing a single scandir() on the containing directory.

That said, I do question just how useful or widely-used it would be. 
How many people (1) have images which are stored without appropriate 
file extensions, and (2) use, or would use, geeqie rather than something 
less powerful/more user-friendly?  Given how many dark, unmaintained 
corners geeqie has already, I definitely think we should make sure that 
the features we add have sufficient value.

--xsdg



------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to