Replace 'probe' with 'channel' in most places.

[libsigrok.git] / output / text / ascii.c

diff --git a/output/text/ascii.c b/output/text/ascii.c
index 3b4f87d8d99d3fe028068c16315e51199737635f..4a6b4f08a920db9ee0e54fec76a9176f510f3312 100644 (file)
--- a/output/text/ascii.c
+++ b/output/text/ascii.c
@@ -1,5 +1,5 @@
 /*
- * This file is part of the sigrok project.
+ * This file is part of the libsigrok project.
  *
  * Copyright (C) 2010-2012 Bert Vermeulen <bert@biot.com>
  * Copyright (C) 2011 Håvard Espeland <gus@ping.uio.no>
@@ -21,24 +21,26 @@
 #include <stdlib.h>
 #include <string.h>
 #include <glib.h>
-#include "sigrok.h"
-#include "sigrok-internal.h"
+#include "libsigrok.h"
+#include "libsigrok-internal.h"
 #include "text.h"
 
+#define LOG_PREFIX "output/ascii"
+
 SR_PRIV int init_ascii(struct sr_output *o)
 {
        return init(o, DEFAULT_BPL_ASCII, MODE_ASCII);
 }
 
-SR_PRIV int data_ascii(struct sr_output *o, const char *data_in,
-                      uint64_t length_in, char **data_out,
+SR_PRIV int data_ascii(struct sr_output *o, const uint8_t *data_in,
+                      uint64_t length_in, uint8_t **data_out,
                       uint64_t *length_out)
 {
        struct context *ctx;
        unsigned int outsize, offset, p;
        int max_linelen;
-       uint64_t sample;
-       char *outbuf;
+       const uint8_t *sample;
+       uint8_t *outbuf;
 
        ctx = o->internal;
        max_linelen = SR_MAX_PROBENAME_LEN + 3 + ctx->samples_per_line
@@ -48,17 +50,17 @@ SR_PRIV int data_ascii(struct sr_output *o, const char *data_in,
          * extra output, e.g. trigger.
          */
        outsize = 512 + (1 + (length_in / ctx->unitsize) / ctx->samples_per_line)
-            * (ctx->num_enabled_probes * max_linelen);
+            * (ctx->num_enabled_channels * max_linelen);
 
        if (!(outbuf = g_try_malloc0(outsize + 1))) {
-               sr_err("ascii out: %s: outbuf malloc failed", __func__);
+               sr_err("%s: outbuf malloc failed", __func__);
                return SR_ERR_MALLOC;
        }
 
        outbuf[0] = '\0';
        if (ctx->header) {
                /* The header is still here, this must be the first packet. */
-               strncpy(outbuf, ctx->header, outsize);
+               strncpy((char *)outbuf, ctx->header, outsize);
                g_free(ctx->header);
                ctx->header = NULL;
        }
@@ -66,14 +68,14 @@ SR_PRIV int data_ascii(struct sr_output *o, const char *data_in,
        if (length_in >= ctx->unitsize) {
                for (offset = 0; offset <= length_in - ctx->unitsize;
                     offset += ctx->unitsize) {
-                       memcpy(&sample, data_in + offset, ctx->unitsize);
+                       sample = data_in + offset;
 
-                       char tmpval[ctx->num_enabled_probes];
+                       char tmpval[ctx->num_enabled_channels];
 
-                       for (p = 0; p < ctx->num_enabled_probes; p++) {
-                               uint64_t curbit = (sample & ((uint64_t) 1 << p));
-                               uint64_t prevbit = (ctx->prevsample &
-                                               ((uint64_t) 1 << p));
+                       for (p = 0; p < ctx->num_enabled_channels; p++) {
+                               uint8_t curbit = (sample[p / 8] & ((uint8_t) 1 << (p % 8)));
+                               uint8_t prevbit = (ctx->prevsample[p / 8] &
+                                               ((uint8_t) 1 << (p % 8)));
 
                                if (curbit < prevbit && ctx->line_offset > 0) {
                                        ctx->linebuf[p * ctx->linebuf_len +
@@ -97,7 +99,7 @@ SR_PRIV int data_ascii(struct sr_output *o, const char *data_in,
                                ctx->mark_trigger = -1;
                        }
 
-                       for (p = 0; p < ctx->num_enabled_probes; p++) {
+                       for (p = 0; p < ctx->num_enabled_channels; p++) {
                                ctx->linebuf[p * ctx->linebuf_len +
                                             ctx->line_offset] = tmpval[p];
                        }
@@ -105,15 +107,14 @@ SR_PRIV int data_ascii(struct sr_output *o, const char *data_in,
                        ctx->line_offset++;
                        ctx->spl_cnt++;
 
-                       ctx->prevsample = sample;
+                       memcpy(ctx->prevsample, sample, ctx->unitsize);
                }
        } else {
-               sr_info("ascii out: short buffer (length_in=%" PRIu64 ")",
-                       length_in);
+               sr_info("Short buffer (length_in=%" PRIu64 ").", length_in);
        }
 
        *data_out = outbuf;
-       *length_out = strlen(outbuf);
+       *length_out = strlen((const char *)outbuf);
 
        return SR_OK;
 }
@@ -125,4 +126,5 @@ SR_PRIV struct sr_output_format output_text_ascii = {
        .init = init_ascii,
        .data = data_ascii,
        .event = event,
+       .cleanup = text_cleanup,
 };