* Melchior FRANZ -- Monday 04 December 2006 15:53:
> * Stefan Seifert -- Monday 04 December 2006 15:47:
> > <foo include="bar.xml" omit-tag="yes"/>
>
> Yes, that's probably the cleanest solution. (Using "y" instead of or
> in addition to "yes", to be consistent with other attribute values.)
My current favorite is "omit-node". The syntax would be:
<whatever include="some-file.xml" omit-node="y"/>
but this would work, too, and do what one would expect:
<whatever include="some-file.xml" omit-node="y">
<something>bad</something>
</whatever>
The contents of some-file.xml as well as the <something> node
would appear on the same level where <whatever> was, as that
node was simply dropped. The <whatever> is completely arbitrary.
One could have written:
<_ include="some-file.xml" omit-node="y"/>
or
<imported-instrument-animations include="some-file.xml" omit-node="y"/>
The changes are less intrusive than I had feared:
diff -u -p -r1.14 props_io.cxx
--- props_io.cxx 8 Mar 2006 18:16:09 -0000 1.14
+++ props_io.cxx 4 Dec 2006 18:34:18 -0000
@@ -229,6 +229,21 @@ PropsVisitor::startElement (const char *
} catch (sg_io_exception &e) {
setException(e);
}
+
+ const char *omit = atts.getValue("omit-node");
+ if (omit && !strcmp(omit, "y")) {
+ int nChildren = node->nChildren();
+ for (int i = 0; i < nChildren; i++) {
+ SGPropertyNode *src = node->getChild(i);
+ const char *name = src->getName();
+ int index = st.counters[name];
+ st.counters[name]++;
+ SGPropertyNode *dst = st.node->getChild(name, index, true);
+ copyProperties(src, dst);
+ }
+ st.node->removeChild(node->getName(), node->getIndex(), false);
+ node = st.node;
+ }
}
const char *type = atts.getValue("type");
Does anyone have objections to the whole idea? Or not understand
what it's good for? Or better suggestions for "omit-node"?
m.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Flightgear-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/flightgear-devel
