X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=src%2Foutput%2Fanalog.c;h=5f77a59c2c3393c1160ea610a116e893f50cf512;hb=28c95cc6c45ede9ec9d184d606bfc16505834429;hp=88b85d18eddf81b489c011a94918ecb9efb30da5;hpb=222fdfd526fde6d8450067675679eca6cc5d211b;p=libsigrok.git

diff --git a/src/output/analog.c b/src/output/analog.c
index 88b85d18..5f77a59c 100644
--- a/src/output/analog.c
+++ b/src/output/analog.c
@@ -31,6 +31,7 @@ struct context {
 	int num_enabled_channels;
 	GPtrArray *channellist;
 	int digits;
+	float *fdata;
 };
 
 enum {
@@ -64,227 +65,21 @@ static int init(struct sr_output *o, GHashTable *options)
 		g_ptr_array_add(ctx->channellist, ch->name);
 		ctx->num_enabled_channels++;
 	}
+	ctx->fdata = NULL;
 
 	return SR_OK;
 }
 
-static void si_printf(float value, GString *out, char *unitstr)
-{
-	float v;
-
-	if (signbit(value))
-		v = -(value);
-	else
-		v = value;
-
-	if (v < 1e-12 || v > 1e+12)
-		g_string_append_printf(out, "%f %s", value, unitstr);
-	else if (v > 1e+9)
-		g_string_append_printf(out, "%f G%s", value / 1e+9, unitstr);
-	else if (v > 1e+6)
-		g_string_append_printf(out, "%f M%s", value / 1e+6, unitstr);
-	else if (v > 1e+3)
-		g_string_append_printf(out, "%f k%s", value / 1e+3, unitstr);
-	else if (v < 1e-9)
-		g_string_append_printf(out, "%f n%s", value * 1e+9, unitstr);
-	else if (v < 1e-6)
-		g_string_append_printf(out, "%f u%s", value * 1e+6, unitstr);
-	else if (v < 1e-3)
-		g_string_append_printf(out, "%f m%s", value * 1e+3, unitstr);
-	else
-		g_string_append_printf(out, "%f %s", value, unitstr);
-
-}
-
-/* Please use the same order as in enum sr_unit (libsigrok.h). */
-static void fancyprint(int unit, int mqflags, float value, GString *out)
-{
-	switch (unit) {
-	case SR_UNIT_VOLT:
-		si_printf(value, out, "V");
-		break;
-	case SR_UNIT_AMPERE:
-		si_printf(value, out, "A");
-		break;
-	case SR_UNIT_OHM:
-		si_printf(value, out, "");
-		g_string_append_unichar(out, 0x2126);
-		break;
-	case SR_UNIT_FARAD:
-		si_printf(value, out, "F");
-		break;
-	case SR_UNIT_KELVIN:
-		si_printf(value, out, "K");
-		break;
-	case SR_UNIT_CELSIUS:
-		si_printf(value, out, "");
-		g_string_append_unichar(out, 0x00b0);
-		g_string_append_c(out, 'C');
-		break;
-	case SR_UNIT_FAHRENHEIT:
-		si_printf(value, out, "");
-		g_string_append_unichar(out, 0x00b0);
-		g_string_append_c(out, 'F');
-		break;
-	case SR_UNIT_HERTZ:
-		si_printf(value, out, "Hz");
-		break;
-	case SR_UNIT_PERCENTAGE:
-		g_string_append_printf(out, "%f %%", value);
-		break;
-	case SR_UNIT_BOOLEAN:
-		if (value > 0)
-			g_string_append_printf(out, "TRUE");
-		else
-			g_string_append_printf(out, "FALSE");
-		break;
-	case SR_UNIT_SECOND:
-		si_printf(value, out, "s");
-		break;
-	case SR_UNIT_SIEMENS:
-		si_printf(value, out, "S");
-		break;
-	case SR_UNIT_DECIBEL_MW:
-		si_printf(value, out, "dBu");
-		break;
-	case SR_UNIT_DECIBEL_VOLT:
-		si_printf(value, out, "dBV");
-		break;
-	case SR_UNIT_UNITLESS:
-		si_printf(value, out, "");
-		break;
-	case SR_UNIT_DECIBEL_SPL:
-		if (mqflags & SR_MQFLAG_SPL_FREQ_WEIGHT_A)
-			si_printf(value, out, "dB(A)");
-		else if (mqflags & SR_MQFLAG_SPL_FREQ_WEIGHT_C)
-			si_printf(value, out, "dB(C)");
-		else if (mqflags & SR_MQFLAG_SPL_FREQ_WEIGHT_Z)
-			si_printf(value, out, "dB(Z)");
-		else
-			/* No frequency weighting, or non-standard "flat" */
-			si_printf(value, out, "dB(SPL)");
-		if (mqflags & SR_MQFLAG_SPL_TIME_WEIGHT_S)
-			g_string_append(out, " S");
-		else if (mqflags & SR_MQFLAG_SPL_TIME_WEIGHT_F)
-			g_string_append(out, " F");
-		if (mqflags & SR_MQFLAG_SPL_LAT)
-			g_string_append(out, " LAT");
-		else if (mqflags & SR_MQFLAG_SPL_PCT_OVER_ALARM)
-			/* Not a standard function for SLMs, so this is
-			 * a made-up notation. */
-			g_string_append(out, " %oA");
-		break;
-	case SR_UNIT_CONCENTRATION:
-		g_string_append_printf(out, "%f ppm", value * (1000 * 1000));
-		break;
-	case SR_UNIT_REVOLUTIONS_PER_MINUTE:
-		si_printf(value, out, "RPM");
-		break;
-	case SR_UNIT_VOLT_AMPERE:
-		si_printf(value, out, "VA");
-		break;
-	case SR_UNIT_WATT:
-		si_printf(value, out, "W");
-		break;
-	case SR_UNIT_WATT_HOUR:
-		si_printf(value, out, "Wh");
-		break;
-	case SR_UNIT_METER_SECOND:
-		si_printf(value, out, "m/s");
-		break;
-	case SR_UNIT_HECTOPASCAL:
-		si_printf(value, out, "hPa");
-		break;
-	case SR_UNIT_HUMIDITY_293K:
-		si_printf(value, out, "%rF");
-		break;
-	case SR_UNIT_DEGREE:
-		si_printf(value, out, "");
-		g_string_append_unichar(out, 0x00b0);
-		break;
-	case SR_UNIT_HENRY:
-		si_printf(value, out, "H");
-		break;
-	case SR_UNIT_GRAM:
-		si_printf(value, out, "g");
-		break;
-	case SR_UNIT_CARAT:
-		si_printf(value, out, "ct");
-		break;
-	case SR_UNIT_OUNCE:
-		si_printf(value, out, "oz");
-		break;
-	case SR_UNIT_TROY_OUNCE:
-		si_printf(value, out, "oz t");
-		break;
-	case SR_UNIT_POUND:
-		si_printf(value, out, "lb");
-		break;
-	case SR_UNIT_PENNYWEIGHT:
-		si_printf(value, out, "dwt");
-		break;
-	case SR_UNIT_GRAIN:
-		si_printf(value, out, "gr");
-		break;
-	case SR_UNIT_TAEL:
-		si_printf(value, out, "tael");
-		break;
-	case SR_UNIT_MOMME:
-		si_printf(value, out, "momme");
-		break;
-	case SR_UNIT_TOLA:
-		si_printf(value, out, "tola");
-		break;
-	case SR_UNIT_PIECE:
-		si_printf(value, out, "pcs");
-		break;
-	default:
-		si_printf(value, out, "");
-		break;
-	}
-
-	/* Please use the same order as in enum sr_mqflag (libsigrok.h). */
-	if (mqflags & SR_MQFLAG_AC)
-		g_string_append_printf(out, " AC");
-	if (mqflags & SR_MQFLAG_DC)
-		g_string_append_printf(out, " DC");
-	if (mqflags & SR_MQFLAG_RMS)
-		g_string_append_printf(out, " RMS");
-	if (mqflags & SR_MQFLAG_DIODE)
-		g_string_append_printf(out, " DIODE");
-	if (mqflags & SR_MQFLAG_HOLD)
-		g_string_append_printf(out, " HOLD");
-	if (mqflags & SR_MQFLAG_MAX)
-		g_string_append_printf(out, " MAX");
-	if (mqflags & SR_MQFLAG_MIN)
-		g_string_append_printf(out, " MIN");
-	if (mqflags & SR_MQFLAG_AUTORANGE)
-		g_string_append_printf(out, " AUTO");
-	if (mqflags & SR_MQFLAG_RELATIVE)
-		g_string_append_printf(out, " REL");
-	/* Note: SR_MQFLAG_SPL_* is handled above. */
-	if (mqflags & SR_MQFLAG_DURATION)
-		g_string_append_printf(out, " DURATION");
-	if (mqflags & SR_MQFLAG_AVG)
-		g_string_append_printf(out, " AVG");
-	if (mqflags & SR_MQFLAG_REFERENCE)
-		g_string_append_printf(out, " REF");
-	if (mqflags & SR_MQFLAG_UNSTABLE)
-		g_string_append_printf(out, " UNSTABLE");
-	g_string_append_c(out, '\n');
-}
-
 static int receive(const struct sr_output *o, const struct sr_datafeed_packet *packet,
 		GString **out)
 {
 	struct context *ctx;
-	const struct sr_datafeed_analog_old *analog_old;
 	const struct sr_datafeed_analog *analog;
 	struct sr_channel *ch;
 	GSList *l;
 	float *fdata;
 	unsigned int i;
-	int num_channels, c, ret, si, digits;
+	int num_channels, c, ret, digits;
 	char *number, *suffix;
 
 	*out = NULL;
@@ -299,26 +94,13 @@ static int receive(const struct sr_output *o, const struct sr_datafeed_packet *p
 	case SR_DF_FRAME_END:
 		*out = g_string_new("FRAME-END\n");
 		break;
-	case SR_DF_ANALOG_OLD:
-		analog_old = packet->payload;
-		fdata = (float *)analog_old->data;
-		*out = g_string_sized_new(512);
-		num_channels = g_slist_length(analog_old->channels);
-		for (si = 0; si < analog_old->num_samples; si++) {
-			for (l = analog_old->channels, c = 0; l; l = l->next, c++) {
-				ch = l->data;
-				g_string_append_printf(*out, "%s: ", ch->name);
-				fancyprint(analog_old->unit, analog_old->mqflags,
-						fdata[si * num_channels + c], *out);
-			}
-		}
-		break;
 	case SR_DF_ANALOG:
 		analog = packet->payload;
 		num_channels = g_slist_length(analog->meaning->channels);
-		if (!(fdata = g_try_malloc(
+		if (!(fdata = g_try_realloc(ctx->fdata,
 						analog->num_samples * num_channels * sizeof(float))))
 			return SR_ERR_MALLOC;
+		ctx->fdata = fdata;
 		if ((ret = sr_analog_to_float(analog, fdata)) != SR_OK)
 			return ret;
 		*out = g_string_sized_new(512);
@@ -336,7 +118,7 @@ static int receive(const struct sr_output *o, const struct sr_datafeed_packet *p
 			for (l = analog->meaning->channels, c = 0; l; l = l->next, c++) {
 				ch = l->data;
 				g_string_append_printf(*out, "%s: ", ch->name);
-				number = g_strdup_printf("%.*f", digits,
+				number = g_strdup_printf("%.*f", MAX(digits, 0),
 						fdata[i * num_channels + c]);
 				g_string_append(*out, number);
 				g_free(number);
@@ -352,21 +134,6 @@ static int receive(const struct sr_output *o, const struct sr_datafeed_packet *p
 	return SR_OK;
 }
 
-static int cleanup(struct sr_output *o)
-{
-	struct context *ctx;
-
-	if (!o || !o->sdi)
-		return SR_ERR_ARG;
-	ctx = o->priv;
-
-	g_ptr_array_free(ctx->channellist, 1);
-	g_free(ctx);
-	o->priv = NULL;
-
-	return SR_OK;
-}
-
 static struct sr_option options[] = {
 	{ "digits", "Digits", "Digits to show", NULL, NULL },
 	ALL_ZERO
@@ -385,6 +152,24 @@ static const struct sr_option *get_options(void)
 	return options;
 }
 
+static int cleanup(struct sr_output *o)
+{
+	struct context *ctx;
+
+	if (!o || !o->sdi)
+		return SR_ERR_ARG;
+	ctx = o->priv;
+
+	g_ptr_array_free(ctx->channellist, 1);
+	g_variant_unref(options[0].def);
+	g_slist_free_full(options[0].values, (GDestroyNotify)g_variant_unref);
+	g_free(ctx->fdata);
+	g_free(ctx);
+	o->priv = NULL;
+
+	return SR_OK;
+}
+
 SR_PRIV struct sr_output_module output_analog = {
 	.id = "analog",
 	.name = "Analog",