[Development] QStorageInfo

Thiago Macieira thiago.macieira at intel.com
Tue Aug 26 18:43:48 CEST 2014


On Tuesday 26 August 2014 09:22:32 Rutledge Shawn wrote:
> After the harrowing review process (92 rounds of patches over 9 months!) the
> VolumeTypeFlag didn't end up in the final patch.  It's a credit to Ivan
> that he had the patience to finally get the current form of QStorageInfo
> into 5.4 branch at all.  But obviously some users have a need to
> distinguish network drives from local ones anyway, so IMO there's nothing
> wrong with writing another patch to make this info available again.  Of
> course trying to distinguish all the other types: internal, removable,
> optical, and RAM drives, would probably be futile on many platforms.  We
> have several bools already, so how about adding a bool isRemote() accessor?

The problem is that we can't reliably give that information. For example, fuse 
filesystems on Linux may be both local and remote. What's worse, some local 
fuse backends may provide worse API than some remote filesystems (locking, 
etc.)

Unless we want to make this a tri-state: definitely local, definitely remote, 
could be either.

I'll block any patch that tries to give a certainty where the lower level 
doesn't give us certainty.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center




More information about the Development mailing list