On 10/5/2015 10:01 PM, David Gibson wrote:
> On Fri, Oct 02, 2015 at 09:55:17PM -0700, Frank Rowand wrote:
>
> Uh.. the actual patch seems to be missing.
>
Ah, the fun of broken email tools. I'll be changing my email tools
in the near future.
This version of the patch is intended to show what is involved
in creating the short input, given the changes to the other
patches. If it looks reasonable, I will add the modifications
to --annotate to allow a choice of either short or long format.
This version also has a totally awkward:
srcpost_string_short()
{
if (pos)
...
if (!pos)
...
else
...
which I will fix in the next version.
-Frank
From: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx>
Show only the current line number in each source annotation instead
of the full range of the object.
TODO:
- modify "--annotate" command line option to choose short or long annotation
Not-signed-off-by: Frank Rowand <frank.rowand@xxxxxxxxxxxxxx>
---
srcpos.c | 34 ++++++++++++++++++++++++++++++++++
srcpos.h | 4 ++++
treesource.c | 8 ++++----
3 files changed, 42 insertions(+), 4 deletions(-)
Index: b/srcpos.c
===================================================================
--- a/srcpos.c
+++ b/srcpos.c
@@ -327,6 +327,40 @@ srcpos_string(struct srcpos *pos)
return pos_str;
}
+char *
+srcpos_string_short(struct srcpos *pos, int first_line)
+{
+ const char *fname = "<no-file>";
+ char *pos_str;
+ int rc;
+
+ if (pos)
+ fname = pos->file->name;
+
+ if (!pos)
+ rc = asprintf(&pos_str, "%s:<no-line>", fname);
+ else
+ rc = asprintf(&pos_str, "%s:%d", fname,
+ (first_line) ? pos->first_line: pos->last_line);
+
+ if (rc == -1)
+ die("Couldn't allocate in srcpos_string_short");
+
+ return pos_str;
+}
+
+char *
+srcpos_string_first(struct srcpos *pos)
+{
+ return srcpos_string_short(pos, 1);
+}
+
+char *
+srcpos_string_last(struct srcpos *pos)
+{
+ return srcpos_string_short(pos, 0);
+}
+
void srcpos_verror(struct srcpos *pos, const char *prefix,
const char *fmt, va_list va)
{
Index: b/treesource.c
===================================================================
--- a/treesource.c
+++ b/treesource.c
@@ -204,7 +204,7 @@ static void write_propval(FILE *f, struc
if (len == 0) {
if (annotate) {
- srcstr = srcpos_string(prop->srcpos);
+ srcstr = srcpos_string_first(prop->srcpos);
fprintf(f, "; /* %s */\n", srcstr);
free(srcstr);
} else {
@@ -238,7 +238,7 @@ static void write_propval(FILE *f, struc
}
if (annotate) {
- srcstr = srcpos_string(prop->srcpos);
+ srcstr = srcpos_string_first(prop->srcpos);
fprintf(f, "; /* %s */\n", srcstr);
free(srcstr);
} else {
@@ -262,7 +262,7 @@ static void write_tree_source_node(FILE
else
fprintf(f, "/ {");
if (annotate) {
- srcstr = srcpos_string(tree->srcpos);
+ srcstr = srcpos_string_first(tree->srcpos);
fprintf(f, " /* %s */\n", srcstr);
free(srcstr);
} else {
@@ -282,7 +282,7 @@ static void write_tree_source_node(FILE
}
write_prefix(f, level);
if (annotate) {
- srcstr = srcpos_string(tree->srcpos);
+ srcstr = srcpos_string_last(tree->srcpos);
fprintf(f, "}; /* %s */\n", srcstr);
free(srcstr);
} else {
Index: b/srcpos.h
===================================================================
--- a/srcpos.h
+++ b/srcpos.h
@@ -107,6 +107,10 @@ extern struct srcpos *srcpos_copy(struct
extern struct srcpos *srcpos_copy_all(struct srcpos *pos);
extern struct srcpos *srcpos_combine(struct srcpos *left_srcpos, struct srcpos *right_srcpos);
extern char *srcpos_string(struct srcpos *pos);
+extern char *srcpos_string_short(struct srcpos *pos, int line);
+extern char *srcpos_string_first(struct srcpos *pos);
+extern char *srcpos_string_last(struct srcpos *pos);
+
extern void srcpos_dump(struct srcpos *pos);
extern void srcpos_verror(struct srcpos *pos, const char *prefix,
--
To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
