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