diff options
author | Lorenz Meier <lm@inf.ethz.ch> | 2012-11-15 17:19:52 +0100 |
---|---|---|
committer | Lorenz Meier <lm@inf.ethz.ch> | 2012-11-15 17:19:52 +0100 |
commit | 7f916779df3d758ade6836adb6e5f66b5b41bdd3 (patch) | |
tree | b99157421c1906647bfe2dabb91354bcb210209b /apps/systemlib | |
parent | df5e4d19042985bd567845dfa464170c169829b4 (diff) | |
download | px4-firmware-7f916779df3d758ade6836adb6e5f66b5b41bdd3.tar.gz px4-firmware-7f916779df3d758ade6836adb6e5f66b5b41bdd3.tar.bz2 px4-firmware-7f916779df3d758ade6836adb6e5f66b5b41bdd3.zip |
Minor cleanup of param load / store
Diffstat (limited to 'apps/systemlib')
-rw-r--r-- | apps/systemlib/param/param.c | 4 | ||||
-rw-r--r-- | apps/systemlib/param/param.h | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/apps/systemlib/param/param.c b/apps/systemlib/param/param.c index ddf9b0975..ebb72ca3e 100644 --- a/apps/systemlib/param/param.c +++ b/apps/systemlib/param/param.c @@ -517,13 +517,11 @@ param_save_default(void) } int result = param_export(fd, false); - /* should not be necessary, over-careful here */ - fsync(fd); close(fd); if (result != 0) { - unlink(param_get_default_file()); warn("error exporting parameters to '%s'", param_get_default_file()); + unlink(param_get_default_file()); return -2; } diff --git a/apps/systemlib/param/param.h b/apps/systemlib/param/param.h index 6fa73b5a4..084cd931a 100644 --- a/apps/systemlib/param/param.h +++ b/apps/systemlib/param/param.h @@ -250,7 +250,7 @@ __EXPORT int param_set_default_file(const char* filename); * a result of a call to param_set_default_file, or the * built-in default. */ -__EXPORT const char *param_get_default_file(void); +__EXPORT const char* param_get_default_file(void); /** * Save parameters to the default file. |