[PATCH] Some infrastructure work for virtual file system

Manuel Klimek klimek at google.com
Tue Feb 11 10:40:11 PST 2014


On Tue, Feb 11, 2014 at 6:49 PM, Ben Langmuir <blangmuir at apple.com> wrote:

>
> > One more thing: is there a specific reason to have the is_*(Status)
> methods all static on the AbstractFileSystem instead of making them methods
> on Status?
>
> Just to parallel the fact that they are global in sys::fs.
>

I'd vote to pull them into members; mainly because it now is in a class
context, and it keeps the "main" interface smaller, and thus easier to
understand at a glance.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140211/6230c46e/attachment.html>


More information about the cfe-commits mailing list