Daniel That is the thing, they both live in lsst::qserv::xrdfs, and so if I remove the "fs" around one of them, things don't compile. I am happy with using the FsFileValidator name (will you do it or should I?) Jacek On 03/27/2014 12:55 PM, Wang, Daniel Liwei wrote: > Jacek, > > On 03/25/2014 09:03 PM, Becla, Jacek wrote: >> I noticed in xrdfs we have the following pattern: >> >> namespace fs { >> class FileValidator { >> virtual void whatever() = 0; >> }; >> } // end of namespace >> >> class FileValidator : public fs::FileValidator { >> virtual void whatever() {} >> }; >> >> Why not simply: >> >> class FileValidatorBase { >> virtual void whatever() = 0; >> }; >> >> class FileValidator : public FileValidatorBase { >> virtual void whatever() {} >> }; > The "Base" suffix looks ugly to me. Because you've cleaned up the > namespaces, we don't need "fs" any more because you now have "xrdfs", > right? The ones in MySqlFs.cc can be FsFileValidator and ChunkInvValidator. > > -Daniel > ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the QSERV-L list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1