param & flashparam param_export: use value directly instead of calling param_get

This call is not needed, and will avoid deadlocks when locking is enabled.
This commit is contained in:
Beat Küng
2017-02-16 17:43:30 +01:00
committed by Lorenz Meier
parent 42967df63f
commit fa3a6b890c
2 changed files with 4 additions and 4 deletions
@@ -107,7 +107,7 @@ param_export_internal(bool only_unsaved)
switch (param_type(s->param)) {
case PARAM_TYPE_INT32:
param_get(s->param, &i);
i = s->val.i;
if (bson_encoder_append_int(&encoder, param_name(s->param), i)) {
debug("BSON append failed for '%s'", param_name(s->param));
@@ -117,7 +117,7 @@ param_export_internal(bool only_unsaved)
break;
case PARAM_TYPE_FLOAT:
param_get(s->param, &f);
f = s->val.f;
if (bson_encoder_append_double(&encoder, param_name(s->param), f)) {
debug("BSON append failed for '%s'", param_name(s->param));
+2 -2
View File
@@ -916,7 +916,7 @@ param_export(int fd, bool only_unsaved)
switch (param_type(s->param)) {
case PARAM_TYPE_INT32: {
param_get(s->param, &i);
i = s->val.i;
const char *name = param_name(s->param);
/* lock as short as possible */
@@ -932,7 +932,7 @@ param_export(int fd, bool only_unsaved)
case PARAM_TYPE_FLOAT: {
param_get(s->param, &f);
f = s->val.f;
const char *name = param_name(s->param);
/* lock as short as possible */