Hi David, Thanks for the reminder. I'm working on a different way to generate the tag files, i.e. using a separate function rather than to generate it as the side-effect of writing the documentation. That way the workaround with the 'tagDataWritten' flag is no longer needed.
Regards, Dimitri On 20 Sep 2014, at 3:43 , David Thompson <david.thomp...@kitware.com> wrote: > Hi all, > > I just wanted to make sure that Jeffrey Melville's patch doesn't get dropped; > I was having the same issue (no tag entry for a base class member when a > subclass also documented it) and his patch got the tag file working for me. > > Thanks, > David > >> Below is a patch that fixed the problem on my codebase. I'm not sure if >> it breaks other cases where you actually want to suppress the tag data >> being generated multiple times. I can submit a Github pull request if >> you'd prefer. >> >> Cheers, >> Jeff >> >> From b874223b33d4939c53b42df75ce62bcc4acea318 Mon Sep 17 00:00:00 2001 >> From: Jeffrey Melville <jmelville@...> >> Date: Mon, 8 Sep 2014 13:16:29 -0400 >> Subject: [PATCH] Fix for missing inherited class members in tag files. >> >> Signed-off-by: Jeffrey Melville <jmelville@...> >> --- >> src/memberdef.cpp | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/src/memberdef.cpp b/src/memberdef.cpp >> index a25528a..8626e5e 100644 >> --- a/src/memberdef.cpp >> +++ b/src/memberdef.cpp >> @@ -3550,7 +3550,11 @@ Specifier MemberDef::virtualness(int count) const >> void MemberDef::_writeTagData(const DefType compoundType) >> { >> unsigned typeMask = 1 << compoundType; >> - if ((m_impl->tagDataWritten) & typeMask) return; // member already >> written for this type >> + >> + // Return if member already written for this compoundType, except if >> + // compoundType is a class. This function can be run multiple times for >> + // classes in an inheritance hierarchy. >> + if (compoundType != TypeClass && (m_impl->tagDataWritten) & typeMask) >> return; >> if (m_impl->mtype==MemberType_EnumValue && m_impl->enumScope && >> m_impl->enumScope->isStrong()) return; // enum value is part of enum >> static bool generateTagFile = >> !Config_getString("GENERATE_TAGFILE").isEmpty(); >> -- >> 1.9.1 > > ------------------------------------------------------------------------------ > Slashdot TV. Video for Nerds. Stuff that Matters. > http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk > _______________________________________________ > Doxygen-users mailing list > Doxygen-users@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/doxygen-users ------------------------------------------------------------------------------ Slashdot TV. Video for Nerds. Stuff that Matters. http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk _______________________________________________ Doxygen-users mailing list Doxygen-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/doxygen-users