/**
* Convert a numeric samplerate value to its "natural" string representation.
*
- * E.g. a value of 3000000 would be converted to "3 MHz", 20000 to "20 kHz".
+ * E.g. a value of 3000000 would be converted to "3 MHz", 20000 to "20 kHz",
+ * 31500 would become "31.5 kHz".
*
* @param samplerate The samplerate in Hz.
- * @return A malloc()ed string representation of the samplerate value,
- * or NULL upon errors. The caller is responsible to free() the memory.
+ *
+ * @return A g_try_malloc()ed string representation of the samplerate value,
+ * or NULL upon errors. The caller is responsible to g_free() the
+ * memory.
*/
SR_API char *sr_samplerate_string(uint64_t samplerate)
{
char *o;
- int r;
-
- o = malloc(30 + 1); /* Enough for a uint64_t as string + " GHz". */
- if (!o)
- return NULL;
+ uint64_t s = samplerate;
- if (samplerate >= SR_GHZ(1))
- r = snprintf(o, 30, "%" PRIu64 " GHz", samplerate / 1000000000);
- else if (samplerate >= SR_MHZ(1))
- r = snprintf(o, 30, "%" PRIu64 " MHz", samplerate / 1000000);
- else if (samplerate >= SR_KHZ(1))
- r = snprintf(o, 30, "%" PRIu64 " kHz", samplerate / 1000);
- else
- r = snprintf(o, 30, "%" PRIu64 " Hz", samplerate);
+ if ((s >= SR_GHZ(1)) && (s % SR_GHZ(1) == 0)) {
+ o = g_strdup_printf("%" PRIu64 " GHz", s / SR_GHZ(1));
+ } else if ((s >= SR_GHZ(1)) && (s % SR_GHZ(1) != 0)) {
+ o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " GHz",
+ s / SR_GHZ(1), s % SR_GHZ(1));
+ } else if ((s >= SR_MHZ(1)) && (s % SR_MHZ(1) == 0)) {
+ o = g_strdup_printf("%" PRIu64 " MHz", s / SR_MHZ(1));
+ } else if ((s >= SR_MHZ(1)) && (s % SR_MHZ(1) != 0)) {
+ o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " MHz",
+ s / SR_MHZ(1), s % SR_MHZ(1));
+ } else if ((s >= SR_KHZ(1)) && (s % SR_KHZ(1) == 0)) {
+ o = g_strdup_printf("%" PRIu64 " kHz", s / SR_KHZ(1));
+ } else if ((s >= SR_KHZ(1)) && (s % SR_KHZ(1) != 0)) {
+ o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " kHz",
+ s / SR_KHZ(1), s % SR_KHZ(1));
+ } else {
+ o = g_strdup_printf("%" PRIu64 " Hz", s);
+ }
- if (r < 0) {
- /* Something went wrong... */
- free(o);
+ if (!o) {
+ sr_err("strutil: %s: Error creating samplerate string.",
+ __func__);
return NULL;
}
* E.g. a value of 3000000 would be converted to "3 us", 20000 to "50 ms".
*
* @param frequency The frequency in Hz.
- * @return A malloc()ed string representation of the frequency value,
- * or NULL upon errors. The caller is responsible to free() the memory.
+ *
+ * @return A g_try_malloc()ed string representation of the frequency value,
+ * or NULL upon errors. The caller is responsible to g_free() the
+ * memory.
*/
SR_API char *sr_period_string(uint64_t frequency)
{
char *o;
int r;
- o = malloc(30 + 1); /* Enough for a uint64_t as string + " ms". */
- if (!o)
+ /* Allocate enough for a uint64_t as string + " ms". */
+ if (!(o = g_try_malloc0(30 + 1))) {
+ sr_err("strutil: %s: o malloc failed", __func__);
return NULL;
+ }
if (frequency >= SR_GHZ(1))
r = snprintf(o, 30, "%" PRIu64 " ns", frequency / 1000000000);
if (r < 0) {
/* Something went wrong... */
- free(o);
+ g_free(o);
return NULL;
}
/**
* TODO
*
- * @param device TODO
+ * @param dev TODO
* @param triggerstring TODO
+ *
* @return TODO
*/
-SR_API char **sr_parse_triggerstring(struct sr_device *device,
+SR_API char **sr_parse_triggerstring(struct sr_dev *dev,
const char *triggerstring)
{
GSList *l;
char **tokens, **triggerlist, *trigger, *tc, *trigger_types;
gboolean error;
- max_probes = g_slist_length(device->probes);
+ max_probes = g_slist_length(dev->probes);
error = FALSE;
if (!(triggerlist = g_try_malloc0(max_probes * sizeof(char *)))) {
}
tokens = g_strsplit(triggerstring, ",", max_probes);
- trigger_types = device->plugin->get_device_info(0, SR_DI_TRIGGER_TYPES);
+ trigger_types = dev->plugin->dev_info_get(0, SR_DI_TRIGGER_TYPES);
if (trigger_types == NULL)
return NULL;
if (tokens[i][0] < '0' || tokens[i][0] > '9') {
/* Named probe */
probenum = 0;
- for (l = device->probes; l; l = l->next) {
+ for (l = dev->probes; l; l = l->next) {
probe = (struct sr_probe *)l->data;
if (probe->enabled
&& !strncmp(probe->name, tokens[i],
*
* @param sizestring A string containing a (decimal) size value.
* @param size Pointer to uint64_t which will contain the string's size value.
- * @return SR_OK or error code
*
+ * @return SR_OK upon success, SR_ERR upon errors.
*/
SR_API int sr_parse_sizestring(const char *sizestring, uint64_t *size)
{