rshen/c-capnproto
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

compiler: drop outdated find_node comment & IDs

Browse source
This commit is contained in:
David Lamparter 2016-07-20 08:47:41 +02:00
parent c322bc9618
commit 54a7bb7ffb
1 changed files with 0 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

18
compiler/capnpc-c.c
View file

@ -36,10 +36,6 @@ struct node {
struct field *fields; struct field *fields;
}; };
// annotation ids introduced via c++.capnp, see comments in find_node()
static const uint64_t NAMESPACE_ANNOTATION_ID = 0xb9c6f99ebf805f2cull;
static const uint64_t NAME_ANNOTATION_ID = 0xf264a779fef191ceull;
static struct str SRC = STR_INIT, HDR = STR_INIT; static struct str SRC = STR_INIT, HDR = STR_INIT;
static struct capn g_valcapn; static struct capn g_valcapn;
static struct capn_segment g_valseg; static struct capn_segment g_valseg;
@ -51,20 +47,6 @@ static int g_fieldgetset = 0;
static struct capn_tree *g_node_tree; static struct capn_tree *g_node_tree;
static struct node *find_node_mayfail(uint64_t id) { static struct node *find_node_mayfail(uint64_t id) {
/*
* TODO: an Annotation is technically a node (since it can show up in
* a Node's NestedNode list), but a `struct node` is currently configured
* to represent only Nodes. So when processing all nested nodes,
* we need a way to handle entities (like Annotation) which can't be
* represented by a `struct node`.
*
* Current workaround is a hard coded test for the annotations
* introduced via c++.capnp (NAME_ANNOTATION_ID and NAMESPACE_ANNOTATION_ID),
* and to report that we don't have a node associated with them
* (but at least don't fail and stop further processing).
*/
struct node *s = (struct node*) g_node_tree; struct node *s = (struct node*) g_node_tree;
while (s && s->n.id != id) { while (s && s->n.id != id) {
s = (struct node*) s->hdr.link[s->n.id < id]; s = (struct node*) s->hdr.link[s->n.id < id];