issue #11307 C++ 20 module partition: Links to class members not found

Author: doxygen

src/docnode.cpp

@@ -707,7 +707,7 @@ DocRef::DocRef(DocParser *parser,DocNodeVariant *parent,const QCString &target,c
 {
   const Definition  *compound = nullptr;
   QCString     anchor;
-  //printf("DocRef::DocRef(target=%s,context=%s)\n",qPrint(target),qPrint(context));
+  AUTO_TRACE("target='{}',context='{}'",target,context);
   ASSERT(!target.isEmpty());
   m_relPath = parser->context.relPath;
   const SectionInfo *sec = SectionManager::instance().find(parser->context.prefix+target);
@@ -745,6 +745,7 @@ DocRef::DocRef(DocParser *parser,DocNodeVariant *parent,const QCString &target,c
     m_sectionType = sec->type();
     //printf("m_text=%s,m_ref=%s,m_file=%s,type=%d\n",
     //    qPrint(m_text),qPrint(m_ref),qPrint(m_file),m_refType);
+    AUTO_TRACE_EXIT("section");
     return;
   }
   else if (resolveLink(context,target,true,&compound,anchor,parser->context.prefix))
@@ -781,6 +782,7 @@ DocRef::DocRef(DocParser *parser,DocNodeVariant *parent,const QCString &target,c
       m_ref  = compound->getReference();
       //printf("isFile=%d compound=%s (%d)\n",isFile,qPrint(compound->name()),
       //    compound->definitionType());
+      AUTO_TRACE_EXIT("compound");
       return;
     }
     else if (compound && compound->definitionType()==Definition::TypeFile &&
@@ -789,8 +791,13 @@ DocRef::DocRef(DocParser *parser,DocNodeVariant *parent,const QCString &target,c
     {
       m_file = compound->getSourceFileBase();
       m_ref  = compound->getReference();
+      AUTO_TRACE_EXIT("source");
       return;
     }
+    else
+    {
+      AUTO_TRACE_EXIT("compound '{}' not linkable",compound?compound->name():"none");
+    }
   }
   m_text = target;
   warn_doc_error(parser->context.fileName,parser->tokenizer.getLineNr(),"unable to resolve reference to '%s' for \\ref command",

src/moduledef.cpp

@@ -59,8 +59,12 @@ class ModuleDefImpl : public DefinitionMixin<ModuleDef>
     QCString displayName(bool=TRUE) const override { return name(); }
     QCString getOutputFileBase() const override;
     QCString anchor() const override { return ""; }
-    bool isLinkableInProject() const override { return isLinkable() && !isHidden() && !isReference(); }
-    bool isLinkable() const override { return hasDocumentation(); }
+    bool isLinkableInProject() const override {
+      if (m_primaryInterface) return m_primaryInterface->isLinkableInProject();
+      else return isLinkable() && !isHidden() && !isReference(); }
+    bool isLinkable() const override {
+      if (m_primaryInterface) return m_primaryInterface->isLinkable();
+      else return hasDocumentation(); }
     QCString qualifiedName() const override;
     void writeSummaryLinks(OutputList &ol) const override;
 

src/util.cpp

@@ -3135,7 +3135,7 @@ bool resolveLink(/* in */ const QCString &scName,
 
   QCString linkRef=lr;
   QCString linkRefWithoutTemplates = stripTemplateSpecifiersFromScope(linkRef,FALSE);
-  //printf("ResolveLink linkRef=%s\n",qPrint(lr));
+  AUTO_TRACE("scName='{}',ref='{}'",scName,lr);
   const FileDef  *fd = nullptr;
   const GroupDef *gd = nullptr;
   const PageDef  *pd = nullptr;
@@ -3147,6 +3147,7 @@ bool resolveLink(/* in */ const QCString &scName,
   bool ambig = false;
   if (linkRef.isEmpty()) // no reference name!
   {
+    AUTO_TRACE_EXIT("no_ref");
     return FALSE;
   }
   else if ((pd=Doxygen::pageLinkedMap->find(linkRef))) // link to a page
@@ -3162,70 +3163,82 @@ bool resolveLink(/* in */ const QCString &scName,
     {
       *resContext=pd;
     }
+    AUTO_TRACE_EXIT("page");
     return TRUE;
   }
   else if ((si=SectionManager::instance().find(prefix+linkRef)))
   {
     *resContext=si->definition();
     resAnchor = si->label();
+    AUTO_TRACE_EXIT("section");
     return TRUE;
   }
   else if ((pd=Doxygen::exampleLinkedMap->find(linkRef))) // link to an example
   {
     *resContext=pd;
+    AUTO_TRACE_EXIT("example");
     return TRUE;
   }
   else if ((gd=Doxygen::groupLinkedMap->find(linkRef))) // link to a group
   {
     *resContext=gd;
+    AUTO_TRACE_EXIT("group");
     return TRUE;
   }
   else if ((fd=findFileDef(Doxygen::inputNameLinkedMap,linkRef,ambig)) // file link
       && fd->isLinkable())
   {
     *resContext=fd;
+    AUTO_TRACE_EXIT("file");
     return TRUE;
   }
   else if ((cd=getClass(linkRef))) // class link
   {
     *resContext=cd;
     resAnchor=cd->anchor();
+    AUTO_TRACE_EXIT("class");
     return TRUE;
   }
   else if ((cd=getClass(linkRefWithoutTemplates))) // C#/Java generic class link
   {
     *resContext=cd;
     resAnchor=cd->anchor();
+    AUTO_TRACE_EXIT("generic");
     return TRUE;
   }
   else if ((cd=getClass(linkRef+"-p"))) // Obj-C protocol link
   {
     *resContext=cd;
     resAnchor=cd->anchor();
+    AUTO_TRACE_EXIT("protocol");
     return TRUE;
   }
   else if ((cnd=getConcept(linkRef))) // C++20 concept definition
   {
     *resContext=cnd;
     resAnchor=cnd->anchor();
+    AUTO_TRACE_EXIT("concept");
     return TRUE;
   }
   else if ((nd=Doxygen::namespaceLinkedMap->find(linkRef)))
   {
     *resContext=nd;
+    AUTO_TRACE_EXIT("namespace");
     return TRUE;
   }
   else if ((dir=Doxygen::dirLinkedMap->find(FileInfo(linkRef.str()).absFilePath()+"/"))
       && dir->isLinkable()) // TODO: make this location independent like filedefs
   {
     *resContext=dir;
+    AUTO_TRACE_EXIT("directory");
     return TRUE;
   }
   else // probably a member reference
   {
     const MemberDef *md = nullptr;
     bool res = resolveRef(scName,lr,TRUE,resContext,&md);
     if (md) resAnchor=md->anchor();
+    AUTO_TRACE_EXIT("member? res={}",res);
     return res;
   }
 }