mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-01-31 18:13:24 +00:00
Code Issues Projects Releases Packages Wiki Activity

cleanup[1/6]: do not set userid/save_userid_local from storage

Browse source 
This the first of a set of cleanups related to the removal of 2
preferences: save_userid_local and userid. The commits are
ordered so that a sane running state remains, should a
bisect ever lands here.

Here, just read a git or XML logbook including the to be removed
preferences, as existing users can have this data sitting around.
The only thing done here is not to store the possibly read data
for the mentioned preferences.

Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
This commit is contained in:
Jan Mulder 2018-09-11 14:39:25 +02:00 • committed by Dirk Hohndel
parent 62a8bc7391
commit 226161ee44
3 changed files with 4 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
core/load-git.c
View file

@ -831,13 +831,13 @@ static void parse_settings_units(char *line, struct membuffer *str, void *unused
}
static void parse_settings_userid(char *line, struct membuffer *str, void *_unused)
/* Keep this despite removal of the webservice as there are legacy logbook around
* that still have this defined.
*/
{
UNUSED(str);
UNUSED(_unused);
if (line) {
prefs.save_userid_local = true;
set_userid(line);
}
UNUSED(line);
}
static void parse_settings_prefs(char *line, struct membuffer *str, void *unused)
@ -1544,7 +1544,6 @@ static int parse_settings_entry(git_repository *repo, const git_tree_entry *entr
git_blob *blob = git_tree_entry_blob(repo, entry);
if (!blob)
return report_error("Unable to read settings file");
prefs.save_userid_local = false;
for_each_line(blob, settings_parser, NULL);
git_blob_free(blob);
return 0;

9
core/parse-xml.c
View file

@ -944,13 +944,6 @@ static void try_to_fill_sample(struct sample *sample, const char *name, char *bu
nonmatch("sample", name, buf);
}
static void try_to_fill_userid(const char *name, char *buf)
{
UNUSED(name);
if (prefs.save_userid_local)
set_userid(buf);
}
static const char *country, *city;
static void divinglog_place(char *place, uint32_t *uuid)
@ -1382,7 +1375,6 @@ static bool entry(const char *name, char *buf)
report_datafile_version(last_xml_version);
}
if (in_userid) {
try_to_fill_userid(name, buf);
return true;
}
if (in_settings) {
@ -1643,7 +1635,6 @@ int parse_xml_buffer(const char *url, const char *buffer, int size,
if (!doc)
return report_error(translate("gettextFromC", "Failed to parse '%s'"), url);
prefs.save_userid_local = false;
reset_all();
dive_start();
doc = test_xslt_transforms(doc, params);

6
core/parse.c
View file

@ -400,12 +400,6 @@ void divecomputer_end(void)
void userid_start(void)
{
in_userid = true;
//if the xml contains userid, keep saving it.
// don't call the prefs method here as we don't wanna
// actually change the preferences, this is temporary and
// will be reverted when the file finishes.
prefs.save_userid_local = true;
}
void userid_stop(void)