FCL/sftools/depl/doctools: comparison Orb/Doxygen/src/xmlditadocvisitor.cpp

equal deleted inserted replaced

-:42188c7ea2d9
+:82f11024044a
+#include "xmldita.h"
 #include "xmlditadocvisitor.h"
+#include "xmlditatrace.h"
 #include "docparser.h"
 #include "language.h"
 #include "doxygen.h"
 #include "outputgen.h"
 #include "xmlgen.h"
 #include "message.h"
 #include "util.h"
 #include <qfileinfo.h>
 #include "parserintf.h"
+//#define DITA_DOT_HACK_REMOVE_XREFS
+#undef DITA_DOT_HACK_REMOVE_XREFS
+// If 0 there is no <simpletable> support
+// Need to lazily evaluate this so that <simpletable> is only written if
+// there is content in the table
+#define DITA_SIMPLETABLE_SUPPORT 0
 XmlDitaDocVisitor::XmlDitaDocVisitor(XmlStream &s,CodeOutputInterface &ci)
 : DocVisitor(DocVisitor_XML), xmlStream(s), xmlElemStack(s), m_ci(ci), m_insidePre(FALSE), m_hide(FALSE),
 m_insideParamlist(FALSE), paramMap(), paramDict(), currParam()
-{}
+{
+	paramDict.setAutoDelete(true);
+}
 //--------------------------------------
 // visitor functions for leaf nodes
 //--------------------------------------
 void XmlDitaDocVisitor::visit(DocWord *w)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocWord*)", w)
 	if (m_hide) {
 		return;
 	}
 	// Catches normal text (text outside of a tag or command)
 	// and puts it in a "p"
 	write(w->word());
 }
 void XmlDitaDocVisitor::visit(DocLinkedWord *w)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocLinkedWord*)", w)
 	if (m_hide) {
 		return;
 	}
-	if (w->getDefinition() != 0) {
+	//printf("XmlDitaDocVisitor calling startLink() DocLinkedWord=`%s'\n", w->word().data());
-		startLink("", w->getDefinition()->qualifiedName(), "");
+	Definition *d = w->getDefinition();
-	} else {
+	if (0) {
+		QString myName;
+		myName = d->qualifiedName();
+		//printf("XmlDitaDocVisitor calling startLink() DocLinkedWord [name]=`%s'\n", myName.data());
+		startLink("", myName, "");
+	} else {
+		//printf("XmlDitaDocVisitor calling startLink() DocLinkedWord [file]=`%s'\n", w->file().data());
+#if DITA_SUPRESS_NAMESPACE_LINKS
+		if (w->file().find("namespace") != 0) {
+			startLink(w->ref(), w->file(), w->anchor());
+		}
+#else
 		startLink(w->ref(), w->file(), w->anchor());
+#endif
 	}
 	write(w->word());
+#if DITA_SUPRESS_NAMESPACE_LINKS
+	if (w->file().find("namespace") != 0) {
+		endLink();
+	}
+#else
 	endLink();
+#endif
 }
 void XmlDitaDocVisitor::visit(DocWhiteSpace *w)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocWhiteSpace*)", w)
 	if (m_hide) {
 		return;
 	}
 	if (m_insidePre) {
 		write(w->chars());
 	}
 }
 void XmlDitaDocVisitor::visit(DocSymbol *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocSymbol*)", s)
 if (m_hide) {
 	return;
 }
 switch(s->symbol())
 }
 }
 void XmlDitaDocVisitor::visit(DocURL *u)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocURL*)", u)
 	if (m_hide) {
 	  return;
 	}
 	if (u->isEmail()) {
 		startXref(QString("mailto:")+QString(u->url()), u->url());
 	} else {
-		startXref(u->url(), u->url());
+		// Need format attribute
+		AttributeMap myMap;
+		myMap["href"] = u->url();
+		myMap["format"] = "html";
+		push("xref", myMap);
+		write(u->url());
+		//startXref(u->url(), u->url());
 	}
 	endXref();
 }
 void XmlDitaDocVisitor::visit(DocLineBreak *lb)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocLineBreak*)", lb)
 	if (m_hide){
 	  return;
 	}
 	//pushpop("linebreak");
 	//if (lb->parent()->kind() == lb->Kind_Verbatim) {
 	}
 }
 void XmlDitaDocVisitor::visit(DocHorRuler *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHorRuler*)")
 	if (m_hide) {
 	  return;
 	}
 	//pushpop("hruler");
 	xmlStream.comment("hruler not supported by DITA 1.1");
 }
 void XmlDitaDocVisitor::visit(DocStyleChange *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocStyleChange*)", s)
 	if (m_hide) {
 	  return;
 	}
 	switch (s->style())
 	{
 	}
 }
 void XmlDitaDocVisitor::visit(DocVerbatim *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocVerbatim*)", s)
 	if (m_hide) {
 	  return;
 	}
 	switch(s->type())
 	{
 	}
 }
 void XmlDitaDocVisitor::visit(DocAnchor *anc)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocAnchor*)", anc)
 	if (m_hide) {
 	  return;
 	}
 	xmlElemStack.addAttribute("id", anc->file()+"_1"+anc->anchor());
 	//push("anchor", "id", anc->file()+"_1"+anc->anchor());
 	//pop("anchor");
 }
 void XmlDitaDocVisitor::visit(DocInclude *inc)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocInclude*)", inc)
 	if (m_hide) {
 	  return;
 	}
 switch(inc->type())
 {
 }
 }
 void XmlDitaDocVisitor::visit(DocIncOperator *op)
 {
-DITA_DOC_VISITOR_TRACE("visit(DocIncOperator*)",op);
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocIncOperator*)", op)
 if (op->isFirst())
 {
 if (!m_hide) {
 		push("codeblock");
 }
 }
 }
 void XmlDitaDocVisitor::visit(DocFormula *f)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocFormula*)", f)
 	write(f->text());
 #if 0
 	if (m_hide) {
 	  return;
 	}
 #endif
 }
 void XmlDitaDocVisitor::visit(DocIndexEntry *ie)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visit(DocIndexEntry*)", ie)
 	if (m_hide) {
 	  return;
 	}
 	push("indexterm");
 // visitor functions for compound nodes
 //--------------------------------------
 void XmlDitaDocVisitor::visitPre(DocAutoList *l)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocAutoList*)", l)
 	if (m_hide) {
 	  return;
 	}
 	if (l->isEnumList()) {
 		push("ol");
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocAutoList *l)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocAutoList*)", l)
 	if (l->isEnumList()) {
 		visitPostDefault("ol");
 	} else {
 		visitPostDefault("ul");
 	}
 }
 void XmlDitaDocVisitor::visitPre(DocAutoListItem *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocAutoListItem*)")
 	visitPreDefault("li");
 }
 void XmlDitaDocVisitor::visitPost(DocAutoListItem *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocAutoListItem*)")
 	visitPostDefault("li");
 }
 void XmlDitaDocVisitor::visitPre(DocPara *p)
 {
-	if (xmlElemStack.isEmpty() || xmlElemStack.peek().getElemName() != "p") {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocPara*)", p)
+	if (canPushPara()) {
 		visitPreDefault("p");
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocPara *)
 {
-	if (!xmlElemStack.isEmpty() && xmlElemStack.peek().getElemName() == "p") {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocPara*)")
+	if (canPopPara()) {
 		visitPostDefault("p");
 	}
 }
 void XmlDitaDocVisitor::visitPre(DocRoot *)
-{}
+{
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocRoot*)")
+}
 void XmlDitaDocVisitor::visitPost(DocRoot *)
-{}
+{
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocRoot*)")
+}
 void XmlDitaDocVisitor::visitPre(DocSimpleSect *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocSimpleSect*)", s)
 	if (m_hide) {
 	  return;
 	}
 	switch(s->type())
 	{
 		case DocSimpleSect::Pre:
 		case DocSimpleSect::Post:
 		case DocSimpleSect::Invar:
 		case DocSimpleSect::User:
 		case DocSimpleSect::Rcs:
-			if (xmlElemStack.isEmpty() || xmlElemStack.peek().getElemName() != "p") {
+			if (canPushPara()) {
 				push("p");
 			}
 			break;
 		case DocSimpleSect::Note:
 			push("note", "type", "note");
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocSimpleSect *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocSimpleSect*)", s)
 	if (m_hide) {
 	  return;
 	}
 	switch(s->type())
 	{
 			pop("note");
 			break;
 		case DocSimpleSect::Unknown:
 			break;
 		default:
-			if (!xmlElemStack.isEmpty() && xmlElemStack.peek().getElemName() == "p") {
+			if (canPopPara()) {
 				pop("p");
 			}
 			break;
 	}
 //visitPostDefault("simplesect");
 }
 void XmlDitaDocVisitor::visitPre(DocTitle *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocTitle*)")
 	if (!xmlElemStack.isEmpty() && xmlElemStack.peek().getElemName() == "concept") {
 		visitPreDefault("title");
 	} else {
-		if (xmlElemStack.isEmpty() || xmlElemStack.peek().getElemName() != "p") {
+		if (canPushPara()) {
 			visitPreDefault("p");
 		}
 		visitPreDefault("b");
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocTitle *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocTitle*)")
 	if (xmlElemStack.peek().getElemName() == "title") {
 		visitPostDefault("title");
 	} else {
 		visitPostDefault("b");
-		if (!xmlElemStack.isEmpty() && xmlElemStack.peek().getElemName() == "p") {
+		if (canPopPara()) {
 			visitPostDefault("p");
 		}
 	}
 }
 void XmlDitaDocVisitor::visitPre(DocSimpleList *)
 {
-visitPreDefault("ul");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocSimpleList*)")
+	visitPreDefault("ul");
 }
 void XmlDitaDocVisitor::visitPost(DocSimpleList *)
 {
-visitPostDefault("ul");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocSimpleList*)")
+	visitPostDefault("ul");
 }
 void XmlDitaDocVisitor::visitPre(DocSimpleListItem *)
 {
-visitPreDefault("li");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocSimpleListItem*)")
+	visitPreDefault("li");
 }
 void XmlDitaDocVisitor::visitPost(DocSimpleListItem *)
 {
-visitPostDefault("li");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocSimpleListItem*)")
+	visitPostDefault("li");
 }
 void XmlDitaDocVisitor::visitPre(DocSection *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocSection*)", s)
 	// Currently unsupported
 #if 0
 	if (m_hide) {
 	  return;
 	}
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocSection *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocSection*)", s)
 #if 0
 	// The original did not have the if(m_hide) test.
 	// I assume that is an error so visitPostDefault() uses it.
 	QString level;
 	level.setNum(s->level());
 #endif
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlList *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocHtmlList*)", s)
 	if (m_hide) {
 	  return;
 	}
 	if (s->type()==DocHtmlList::Ordered) {
 		push("ol");
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlList *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocHtmlList*)", s)
 	if (s->type()==DocHtmlList::Ordered) {
 		visitPostDefault("ol");
 	} else {
 		visitPostDefault("ul");
 	}
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlListItem *)
 {
-visitPreDefault("li");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlListItem*)")
+	visitPreDefault("li");
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlListItem *)
 {
-visitPostDefault("li");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlListItem*)")
+	visitPostDefault("li");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlDescList *)
 {
-visitPreDefault("dl");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlDescList*)")
+	visitPreDefault("dl");
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlDescList *)
 {
-visitPostDefault("dl");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlDescList*)")
+	visitPostDefault("dl");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlDescTitle *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlDescTitle*)")
 	if (m_hide) {
 	  return;
 	}
 	push("dlentry");
 	push("dt");
 	//push("term");
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlDescTitle *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlDescTitle*)")
 	if (m_hide) {
 	  return;
 	}
 	pop("dt");
 	//pop("dlentry");
 	// pop("varlistentry");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlDescData *)
 {
-push("dd");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlDescData*)")
-//visitPreDefault("li");
+	push("dd");
+	//visitPreDefault("li");
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlDescData *)
 {
-pop("dd");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlDescData*)")
-pop("dlentry");
+	pop("dd");
-//visitPostDefault("li");
+	pop("dlentry");
+	//visitPostDefault("li");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlTable *t)
 {
-	if (m_hide) {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocHtmlTable*)", t)
-	  return;
+	if (m_hide) {
-	}
+	  return;
+	}
+#if DITA_SIMPLETABLE_SUPPORT
 	push("simpletable");
+#endif
 #if 0
 	AttributeMap attrs;
 	QString vR, vC;
 	vR.setNum(t->numRows());
 	attrs["rows"] = vR;
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlTable *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlTable*)")
+#if DITA_SIMPLETABLE_SUPPORT
 	visitPostDefault("simpletable");
+#endif
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlRow *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlRow*)")
 	// FIXME look ahead to first cell
 	// if isHeading is true do
 	// visitPreDefault("sthead");
 	// else
 	visitPreDefault("strow");
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlRow *)
 {
-visitPostDefault("strow");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlRow*)")
+	visitPostDefault("strow");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlCell *c)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocHtmlCell*)", c)
 	visitPreDefault("stentry");
 #if 0
 	if (m_hide) {
 	  return;
 	}
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlCell *c)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocHtmlCell*)", c)
 visitPostDefault("stentry");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlCaption *)
 {
-// Caption is unsupported
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocHtmlCaption*)")
+	// Caption is unsupported
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlCaption *)
 {
-// Caption is unsupported
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlCaption*)")
+	// Caption is unsupported
 }
 void XmlDitaDocVisitor::visitPre(DocInternal *)
 {
-//visitPreDefault("internal");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocInternal*)")
+	//visitPreDefault("internal");
 }
 void XmlDitaDocVisitor::visitPost(DocInternal *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocInternal*)")
 //visitPostDefault("internal");
 }
 void XmlDitaDocVisitor::visitPre(DocHRef *href)
 {
-push("xref", "href", href->url());
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocHRef*)", href)
+	push("xref", "href", href->url());
 }
 void XmlDitaDocVisitor::visitPost(DocHRef *)
 {
-visitPostDefault("xref");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHRef*)")
+	visitPostDefault("xref");
 }
 void XmlDitaDocVisitor::visitPre(DocHtmlHeader *header)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocHtmlHeader*)", header)
 	visitPreDefault("b");
 #if 0
 QString hdgLevel;
 hdgLevel.setNum(header->level());
 push("heading", "level", hdgLevel);
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocHtmlHeader *)
 {
-visitPostDefault("b");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocHtmlHeader*)")
+	visitPostDefault("b");
 }
 void XmlDitaDocVisitor::visitPre(DocImage *img)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocImage*)", img)
 	// Currently unsupported
 #if 0
 AttributeMap imgAttrs;
 // First the image type
 switch(img->type())
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocImage *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocImage*)")
 //visitPostDefault("image");
 }
 void XmlDitaDocVisitor::visitPre(DocDotFile *df)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocDotFile*)", df)
 	// Currently unsupported
 #if 0
 	if (m_hide) {
 	  return;
 	}
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocDotFile *)
 {
-//  visitPostDefault("dotfile");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocDotFile*)")
+	//visitPostDefault("dotfile");
 }
 void XmlDitaDocVisitor::visitPre(DocLink *lnk)
 {
-	if (m_hide) {
+	// The result of a \link...\endlink command
-	  return;
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocLink*)", lnk)
-	}
+	if (m_hide) {
-	if (lnk->getDefinition() != 0) {
+	  return;
+	}
+	if (0) {//lnk->getDefinition() != 0) {
+		//printf("XmlDitaDocVisitor calling startLink() DocLink [name]=`%s'\n", lnk->getDefinition()->qualifiedName().data());
 		startLink("", lnk->getDefinition()->qualifiedName(), "");
 	} else {
-		startLink(lnk->ref(),lnk->file(),lnk->anchor());
+		//printf("XmlDitaDocVisitor calling startLink() DocLink [file]=`%s'\n", lnk->file().data());
+		//startLink(lnk->ref(),lnk->file(),lnk->anchor());
+		startLink(lnk->ref(), lnk->file(), lnk->anchor());
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocLink *)
 {
+	// The result of a \link...\endlink command
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocLink*)")
 	if (m_hide) {
 	  return;
 	}
 	endLink();
 }
 void XmlDitaDocVisitor::visitPre(DocRef *ref)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocRef*)", ref)
 	if (m_hide) {
 	  return;
 	}
 	if (!ref->file().isEmpty()) {
 		if (ref->getDefinition() != 0) {
+			//printf("XmlDitaDocVisitor calling startLink() DocRef [name]=`%s'\n", ref->getDefinition()->qualifiedName().data());
 			startLink("", ref->getDefinition()->qualifiedName(), "");
 		} else {
+			//printf("XmlDitaDocVisitor calling startLink() DocRef [file]=`%s'\n", ref->file().data());
 			startLink(ref->ref(), ref->file(), ref->anchor());
 		}
 	}
 	if (!ref->hasLinkText()) {
 	  write(ref->targetTitle());
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocRef *ref)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPost(DocRef*)", ref)
 	if (m_hide) {
 	  return;
 	}
 	if (!ref->file().isEmpty()) {
 	  endLink();
 	write(" ");
 }
 void XmlDitaDocVisitor::visitPre(DocSecRefItem *ref)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocSecRefItem*)", ref)
 	if (m_hide) {
 	  return;
 	}
 	push("li", "id", ref->file()+"_1"+ref->anchor());
 }
 void XmlDitaDocVisitor::visitPost(DocSecRefItem *)
 {
-visitPostDefault("li");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocSecRefItem*)")
+	visitPostDefault("li");
 }
 void XmlDitaDocVisitor::visitPre(DocSecRefList *)
 {
-visitPreDefault("ul");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocSecRefList*)")
+	visitPreDefault("ul");
 }
 void XmlDitaDocVisitor::visitPost(DocSecRefList *)
 {
-visitPostDefault("ul");
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocSecRefList*)")
+	visitPostDefault("ul");
 }
 void XmlDitaDocVisitor::visitPre(DocParamSect *s)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocParamSect*)", s)
 	m_insideParamlist = TRUE;
 	if (m_hide) {
 	  return;
 	}
 	switch(s->type()) {
 	}
 }
 void XmlDitaDocVisitor::visitPost(DocParamSect *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocParamSect*)")
 	visitPostDefault("paraml");
 	m_insideParamlist = FALSE;
 }
 void XmlDitaDocVisitor::visitPre(DocParamList *pl)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocParamList*)", pl)
 	if (m_hide) {
 	  return;
 	}
 	QListIterator<DocNode> li(pl->parameters());
 	DocNode *param;
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocParamList *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocParamList*)")
 	if (m_hide) {
 	  return;
 	}
 	pop("parameterdescription");
 	pop("parameteritem");
 }
 void XmlDitaDocVisitor::visitPre(DocXRefItem *x)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocXRefItem*)", x)
 	if (m_hide) {
 	  return;
 	}
 	// \deprecated commands result in DocXRefItem
 	// with "deprecated" as the filename
 	if (x->file() == "deprecated"){
 		// Fall through to start new paragraph for deprecated description
-	}else
+	} else {
-		{
+		QString hrefStr = x->file();
-		push("xref", "id", x->file()+"_1"+x->anchor());
+		hrefStr.append(Config_getString("XML_DITA_EXTENSION"));
+		hrefStr.append("#");
+		hrefStr.append(x->file());
+		hrefStr.append("_1");
+		hrefStr.append(x->anchor());
+		push("xref", "href", hrefStr);
 		write(x->title());
 	}
 #if 0
 	push("xrefsect", "id", x->file()+"_1"+x->anchor());
 	pushpop("xreftitle", x->title());
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocXRefItem *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocXRefItem*)")
 	if (m_hide) {
 	  return;
 	}
 	// An xref will be top of stack unless the current
 	// DocXRefItem was caused by a \deprecated command
 #endif
 }
 void XmlDitaDocVisitor::visitPre(DocInternalRef *ref)
 {
-	if (m_hide) {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocInternalRef*)", ref)
-	  return;
+	if (m_hide) {
-	}
+	  return;
+	}
+	//printf("XmlDitaDocVisitor calling startLink() DocInternalRef [file]=`%s'\n", ref->file().data());
 	startLink(0, ref->file(), ref->anchor());
 }
 void XmlDitaDocVisitor::visitPost(DocInternalRef *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocXRefItem*)")
 	if (m_hide) {
 	  return;
 	}
 	endLink();
 	write(" ");
 }
 void XmlDitaDocVisitor::visitPre(DocCopy *c)
 {
+	DITA_DOC_VISITOR_TRACE("XmlDitaDocVisitor::visitPre(DocCopy*)", c)
 	// Currently unsupported
 #if 0
 	if (m_hide) {
 	  return;
 	}
 #endif
 }
 void XmlDitaDocVisitor::visitPost(DocCopy *)
 {
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocCopy*)")
 //	visitPostDefault("copydoc");
 }
 void XmlDitaDocVisitor::visitPre(DocText *)
-{}
+{
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPre(DocText*)")
+}
 void XmlDitaDocVisitor::visitPost(DocText *)
-{}
+{
+	DITA_DOC_VISITOR_TRACE_NOARG("XmlDitaDocVisitor::visitPost(DocText*)")
+}
 void XmlDitaDocVisitor::startXref(const QString &href,const QString &text)
 {
+#ifndef DITA_DOT_HACK_REMOVE_XREFS
 	push("xref", "href", href);
+#endif
 	write(text);
 }
 void XmlDitaDocVisitor::endXref()
 {
+#ifndef DITA_DOT_HACK_REMOVE_XREFS
 	pop("xref");
+#endif
 }
 void XmlDitaDocVisitor::startLink(const QString &ref,const QString &file,const QString &anchor)
 {
 AttributeMap refAttrs;
+/*
+printf("XmlDitaDocVisitor::startLink(): ref: \"%s\", file: \"%s\", anchor: \"%s\"\n",
+	  ref.data(),
+	  file.data(),
+	  anchor.data());
+*/
 if (!anchor.isEmpty()) {
-	  refAttrs["href"] = file+"_1"+anchor;
+	  refAttrs["href"] = file+".xml#"+file+"_1"+anchor;
 } else {
-	  refAttrs["href"] = file;
+	  refAttrs["href"] = file+".xml#"+file;
 }
+#ifndef DITA_DOT_HACK_REMOVE_XREFS
 push("xref", refAttrs);
+#endif
 #if 0
 AttributeMap refAttrs;
 if (!anchor.isEmpty()) {
 	  refAttrs["refid"] = file+"_1"+anchor;
 	  refAttrs["kindref"] = "member";
 #endif
 }
 void XmlDitaDocVisitor::endLink()
 {
+#ifndef DITA_DOT_HACK_REMOVE_XREFS
 visitPostDefault("xref");
+#endif
 }
 void XmlDitaDocVisitor::pushEnabled()
 {
 m_enabled.push(new bool(m_hide));
 		// TODO positional option
 		return "";
 	}
 }
+/// Returns true if it is OK to write a para element
+bool XmlDitaDocVisitor::canPushPara() const
+{
+	if (!xmlElemStack.isEmpty()) {
+		QString e = xmlElemStack.peek().getElemName();
+		if (e == "xref" || e == "p") {
+			return false;
+		}
+	}
+	return true;
+}
+bool XmlDitaDocVisitor::canPopPara() const
+{
+	if (!xmlElemStack.isEmpty() && xmlElemStack.peek().getElemName() == "p") {
+			return true;
+	}
+	return false;
+}
 /** Default treatment of a post traversal visit, this just
 pushes a single element with no attributes. */
 void XmlDitaDocVisitor::visitPreDefault(const QString& elem)
 {
 	if (m_hide) {

changeset 1	82f11024044a
parent 0	42188c7ea2d9