diff options
| author | mvglasow <michael -at- vonglasow.com> | 2018-08-12 15:36:02 +0200 |
|---|---|---|
| committer | mvglasow <michael -at- vonglasow.com> | 2018-08-12 15:36:02 +0200 |
| commit | eab228569770c7ddb900f2f895fc44019ee662ed (patch) | |
| tree | 9a91dfb7392247360e38ebb0d4069e326c006d21 /navit/route.c | |
| parent | 610f91d7b1d5c7c989304584bb77c5d3d01995ef (diff) | |
| download | navit-eab228569770c7ddb900f2f895fc44019ee662ed.tar.gz | |
Refactor:core:Edit some FIXME/TODO comments
Signed-off-by: mvglasow <michael -at- vonglasow.com>
Diffstat (limited to 'navit/route.c')
| -rw-r--r-- | navit/route.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/navit/route.c b/navit/route.c index 12a73a3c7..64d272abe 100644 --- a/navit/route.c +++ b/navit/route.c @@ -2687,7 +2687,9 @@ static int route_graph_is_path_computed(struct route_graph *this_) { * * @param this_ The route */ -// FIXME this is absolutely not thread-safe and will wreak havoc if run concurrently with route_graph_flood() +/* TODO This is absolutely not thread-safe and will wreak havoc if run concurrently with route_graph_flood(). This is + * not an issue as long as the two never overlap: Currently both this function and route_graph_flood() run without + * interruption until they finish, and are both on the main thread. If that changes, we need to revisit this. */ void route_recalculate_partial(struct route *this_) { struct attr route_status; @@ -3502,7 +3504,7 @@ static int rm_attr_get(void *priv_data, enum attr_type attr_type, struct attr *a return 0; return 1; case attr_time: - /* FIXME This ignores access flags on traffic distortions, but the attribute does not seem + /* TODO This ignores access flags on traffic distortions, but the attribute does not seem * to be used anywhere */ mr->attr_next=attr_speed; if (seg) @@ -3511,7 +3513,7 @@ static int rm_attr_get(void *priv_data, enum attr_type attr_type, struct attr *a return 0; return 1; case attr_speed: - /* FIXME This ignores access flags on traffic distortions, but the attribute does not seem + /* TODO This ignores access flags on traffic distortions, but the attribute does not seem * to be used anywhere */ mr->attr_next=attr_label; if (seg) |