odometer reset crash navit

[heiko wegeler]

please add this to navit.xml

<osd enabled="yes" type="odometer" w="350" h="40"  x="30" y="300"   font_size="450" label="Time:${time}" name="persistent_odometer_1" disable_reset="1" autostart="1"/>
<osd enabled="yes" type="text" label="reset" x="200" y="200" command="odometer_reset(&quot;persistent_odometer_1&quot;)"/>

start navit, click reset item -> crash.
with disable_reset="0" the reset with doubleclick on the time item works.

[heiko wegeler]

crash at line 782 in osd_core.c

[tegzed]

Hello Wegeler,

Hopefully I fixed this problem. Please check svn 5062 or a newer build and close ticket if it's OK on your side too.

[heiko wegeler]

Hello KaZeR,

no crashes now, thanks.

But wiki says reset command is for use with disable_reset="1", which was not the case. Moved "if (!this->bDisableReset)" to osd_odometer_click.

and reset command sets odometer inactive. Should untouched. Changes in osd_cmd_odometer_reset to set time to 0.

[antiram]

respects autostart=1

[antiram]

changeset:5224 is only part of a fix. After click reset button, the odometer items ends always in stop mode if autostart=1 is set.

odometer2.diff for r5224 respects autostart=1 (like first odometer.diff)

[antiram]

changeset:5225 is more buggy. After click reset button, the odometer time item is not resetted. You need the lines 527-533 from odometer2.diff somewhere.

Btw, my odometer patch works since 4 month and don't need this unreadable flag in osd_odometer_reset.

[usul]

Summary: Check if the odometer OSD widget behaves as expected.

[heiko wegeler]

no new test required because no change was made in the last 10 month, which addresses this.

I use currently 5532 with odometer changes for this and other bugs and new features for cycling, therefore i can't post an updated patch only for this again.

[http://wiki.navit-project.org/index.php/user:jkoan]

This ticket was pushed back in order to bring 0.5.1 out soon.

[http://wiki.navit-project.org/index.php/user:jkoan]

Ticket retargeted after milestone closed