From: Linus Torvalds Date: Mon, 5 Sep 2011 16:39:55 +0000 (-0700) Subject: Turn dive depth, temperature and duration into xml attributes X-Git-Url: http://git.tdb.fi/?a=commitdiff_plain;h=04c98344b3c50b58dcc62eeb6d13e11030eb662d;p=ext%2Fsubsurface.git Turn dive depth, temperature and duration into xml attributes This makes the xml save-file look way nicer: it's both smaller and better organized. Using individual xml nodes for random small details is silly. The duration even parses exactly the same, because it still ends up being '.depth.duration' (now it's the 'duration' attribute of the dive node, it used to be the 'duration' child node of the dive node). Signed-off-by: Linus Torvalds --- diff --git a/parse-xml.c b/parse-xml.c index f68aae6..393ee00 100644 --- a/parse-xml.c +++ b/parse-xml.c @@ -633,6 +633,10 @@ static void try_to_fill_dive(struct dive *dive, const char *name, char *buf) return; if (MATCH(".meandepth", depth, &dive->meandepth)) return; + if (MATCH(".depth.max", depth, &dive->maxdepth)) + return; + if (MATCH(".depth.mean", depth, &dive->meandepth)) + return; if (MATCH(".duration", duration, &dive->duration)) return; if (MATCH(".divetime", duration, &dive->duration)) @@ -645,6 +649,10 @@ static void try_to_fill_dive(struct dive *dive, const char *name, char *buf) return; if (MATCH(".watertemp", temperature, &dive->watertemp)) return; + if (MATCH(".temperature.air", temperature, &dive->airtemp)) + return; + if (MATCH(".temperature.water", temperature, &dive->watertemp)) + return; if (MATCH(".cylinderstartpressure", pressure, &dive->cylinder[0].start)) return; if (MATCH(".cylinderendpressure", pressure, &dive->cylinder[0].end)) diff --git a/save-xml.c b/save-xml.c index ddee64e..07b8b80 100644 --- a/save-xml.c +++ b/save-xml.c @@ -125,13 +125,32 @@ static void show_utf8(FILE *f, const char *text, const char *pre, const char *po fputs(post, f); } +static void save_depths(FILE *f, struct dive *dive) +{ + /* What's the point of this dive entry again? */ + if (!dive->maxdepth.mm && !dive->meandepth.mm) + return; + + fputs(" maxdepth, " max='", "'"); + show_depth(f, dive->meandepth, " mean='", "'"); + fputs(" />\n", f); +} + +static void save_temperatures(FILE *f, struct dive *dive) +{ + if (!dive->airtemp.mkelvin && !dive->watertemp.mkelvin) + return; + fputs(" airtemp, " air='", "'"); + show_temperature(f, dive->watertemp, " water='", "'"); + fputs(" />\n", f); +} + static void save_overview(FILE *f, struct dive *dive) { - show_depth(f, dive->maxdepth, " ", "\n"); - show_depth(f, dive->meandepth, " ", "\n"); - show_temperature(f, dive->airtemp, " ", "\n"); - show_temperature(f, dive->watertemp, " ", "\n"); - show_duration(f, dive->duration, " ", "\n"); + save_depths(f, dive); + save_temperatures(f, dive); show_duration(f, dive->surfacetime, " ", "\n"); show_utf8(f, dive->location, " ","\n"); show_utf8(f, dive->notes, " ","\n"); @@ -185,9 +204,10 @@ static void save_dive(FILE *f, struct dive *dive) int i; struct tm *tm = gmtime(&dive->when); - fprintf(f, "\n", + fprintf(f, "\n", tm->tm_year+1900, tm->tm_mon+1, tm->tm_mday, - tm->tm_hour, tm->tm_min, tm->tm_sec); + tm->tm_hour, tm->tm_min, tm->tm_sec, + FRACTION(dive->duration.seconds, 60)); save_overview(f, dive); save_cylinder_info(f, dive); for (i = 0; i < dive->samples; i++)