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
|