X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=output%2Foutput_gnuplot.c;h=5d4c564b1071e646f8f9e7c13875b6247f269ca4;hb=340f6e7aeaf0c6cbbb27c5b535787bd09407994b;hp=3a3e7496f08fefea0f023e5eff775a5ad8807c8f;hpb=1e32053cd0a8e42e628cf832d81360b96c8de97d;p=libsigrok.git

diff --git a/output/output_gnuplot.c b/output/output_gnuplot.c
index 3a3e7496..5d4c564b 100644
--- a/output/output_gnuplot.c
+++ b/output/output_gnuplot.c
@@ -36,7 +36,11 @@ const char *gnuplot_header = "\
 #\n\
 # Generated by: %s on %s%s\
 # Timescale: %d %s\n\
-# Column assignment:\n%s\n";
+#\n\
+# Column\tProbe\n\
+# -------------------------------------\
+----------------------------------------\n\
+# 0\t\tSample counter (for internal gnuplot purposes)\n%s\n";
 
 const char *gnuplot_header_comment = "\
 # Comment: Acquisition with %d/%d probes at %s\n";
@@ -63,8 +67,9 @@ static int init(struct output *o)
 	ctx->num_enabled_probes = 0;
 	for (l = o->device->probes; l; l = l->next) {
 		probe = l->data;
-		if (probe->enabled)
-			ctx->probelist[ctx->num_enabled_probes++] = probe->name;
+		if (!probe->enabled)
+			continue;
+		ctx->probelist[ctx->num_enabled_probes++] = probe->name;
 	}
 
 	ctx->probelist[ctx->num_enabled_probes] = 0;
@@ -98,7 +103,7 @@ static int init(struct output *o)
 	wbuf[0] = '\0';
 	for (i = 0; i < ctx->num_enabled_probes; i++) {
 		c = (char *)&wbuf + strlen((char *)&wbuf);
-		sprintf(c, "# Column %d: channel %s\n", i, ctx->probelist[i]);
+		sprintf(c, "# %d\t\t%s\n", i + 1, ctx->probelist[i]);
 	}
 
 	/* TODO: date: File or signals? Make y/n configurable. */
@@ -117,8 +122,6 @@ static int event(struct output *o, int event_type, char **data_out,
 		 uint64_t *length_out)
 {
 	struct context *ctx;
-	char *outbuf;
-	int outlen = 1; /* FIXME */
 
 	ctx = o->internal;
 	switch (event_type) {
@@ -126,11 +129,8 @@ static int event(struct output *o, int event_type, char **data_out,
 		/* TODO */
 		break;
 	case DF_END:
-		outbuf = calloc(1, 1); /* FIXME */
-		if (outbuf == NULL)
-			return SIGROK_ERR_MALLOC;
-		*data_out = outbuf;
-		*length_out = outlen;
+		*data_out = NULL;
+		*length_out = 0;
 		free(o->internal);
 		o->internal = NULL;
 		break;
@@ -144,7 +144,8 @@ static int data(struct output *o, char *data_in, uint64_t length_in,
 {
 	struct context *ctx;
 	unsigned int i, outsize, p, curbit;
-	uint64_t sample, count = 0;
+	uint64_t sample;
+	static uint64_t samplecount = 0;
 	char *outbuf, *c;
 
 	ctx = o->internal;
@@ -171,11 +172,11 @@ static int data(struct output *o, char *data_in, uint64_t length_in,
 
 		/* The first column is a counter (needed for gnuplot). */
 		c = outbuf + strlen(outbuf);
-		sprintf(c, "%" PRIu64 "\t\t", count++);
+		sprintf(c, "%" PRIu64 "\t\t", samplecount++);
 
 		/* The next columns are the values of all channels. */
 		for (p = 0; p < ctx->num_enabled_probes; p++) {
-			curbit = (sample & ((uint64_t) (1 << p))) != 0;
+			curbit = (sample & ((uint64_t) (1 << p))) >> p;
 			c = outbuf + strlen(outbuf);
 			sprintf(c, "%d ", curbit);
 		}