diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2018-11-14 15:24:49 +1100 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2018-11-14 15:24:49 +1100 |
commit | a3143fafbf83868748c99aa14daf274de5549826 (patch) | |
tree | 20ba91881fe53edc26d96ae5b80d0528a924b8e4 /srcpos.c | |
parent | 403cc79f06a135aee00f35cea975f068a72dbd92 (diff) | |
download | device-tree-compiler-a3143fafbf83868748c99aa14daf274de5549826.tar.gz |
Revert "annotations: add positions"
This reverts commit baa1d2cf7894a32bf2f640ef40ebce561b2df565.
Turns out this introduced memory badness. valgrind picks it up on
x86, but it straight out SEGVs on x86.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'srcpos.c')
-rw-r--r-- | srcpos.c | 22 |
1 files changed, 0 insertions, 22 deletions
@@ -207,7 +207,6 @@ struct srcpos srcpos_empty = { .last_line = 0, .last_column = 0, .file = NULL, - .next = NULL, }; void srcpos_update(struct srcpos *pos, const char *text, int len) @@ -235,34 +234,13 @@ struct srcpos * srcpos_copy(struct srcpos *pos) { struct srcpos *pos_new; - struct srcfile_state *srcfile_state; - - if (!pos) - return NULL; pos_new = xmalloc(sizeof(struct srcpos)); memcpy(pos_new, pos, sizeof(struct srcpos)); - /* allocate without free */ - srcfile_state = xmalloc(sizeof(struct srcfile_state)); - memcpy(srcfile_state, pos->file, sizeof(struct srcfile_state)); - pos_new->file = srcfile_state; - return pos_new; } -struct srcpos *srcpos_extend(struct srcpos *pos, struct srcpos *newtail) -{ - struct srcpos *p; - - if (!pos) - return newtail; - - for (p = pos; p->next != NULL; p = p->next); - p->next = newtail; - return pos; -} - char * srcpos_string(struct srcpos *pos) { |