Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-3898

Fix xrootd compiler warnings with clang

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: xrootd
    • Labels:
      None

      Description

      Xrootd

      In file included from core/modules/qdisp/Executive.cc:64:
      In file included from core/modules/qdisp/XrdSsiMocks.h:33:
      In file included from /Users/timj/work/lsstsw/stack/DarwinX86/xrootd/u.timj.DM-3584-ge22410fa7f+da39a3ee5e/include/xrootd/XrdSsi/XrdSsiRequest.hh:37:
      /Users/timj/work/lsstsw/stack/DarwinX86/xrootd/u.timj.DM-3584-ge22410fa7f+da39a3ee5e/include/xrootd/XrdSsi/XrdSsiRespInfo.hh:43:1: warning: 'XrdSsiRespInfo' defined as a
            struct here but previously declared as a class [-Wmismatched-tags]
      struct  XrdSsiRespInfo
      ^
      /Users/timj/work/lsstsw/stack/DarwinX86/xrootd/u.timj.DM-3584-ge22410fa7f+da39a3ee5e/include/xrootd/XrdSsi/XrdSsiSession.hh:45:1: note: did you mean struct here?
      class XrdSsiRespInfo;
      ^~~~~
      struct
      

      core/modules/xrdoss/QservOss.h:64:17: warning: 'lsst::qserv::xrdoss::FakeOssDf::Opendir' hides overloaded virtual function [-Woverloaded-virtual]
          virtual int Opendir(const char *) { return XrdOssOK; }
                      ^
      /Users/timj/work/lsstsw/stack/DarwinX86/xrootd/u.timj.DM-3584-ge22410fa7f+da39a3ee5e/include/xrootd/XrdOss/XrdOss.hh:63:17: note: hidden overloaded virtual function
            'XrdOssDF::Opendir' declared here: different number of parameters (2 vs 1)
      virtual int     Opendir(const char *, XrdOucEnv &)           {return -ENOTDIR;}
                      ^
      

      In file included from core/modules/xrdsvc/SsiSession.h:32:
      /Users/timj/work/lsstsw/stack/DarwinX86/xrootd/u.timj.DM-3584-ge22410fa7f+da39a3ee5e/include/xrootd/XrdSsi/XrdSsiResponder.hh:177:27: warning: control may reach end of
            non-void function [-Wreturn-type]
                                }
                                ^
      

        Attachments

          Container Issues

            Issue Links

              Activity

                People

                • Assignee:
                  fritzm Fritz Mueller
                  Reporter:
                  fritzm Fritz Mueller
                  Reviewers:
                  Andy Hanushevsky, Tim Jenness
                  Watchers:
                  Andy Hanushevsky, Fritz Mueller, Jacek Becla, Tim Jenness
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Summary Panel