search

remulasce / metroapp

Realtime arrival notification utility for LA Metro
2 stars 0 forks source link

Dismissing multiple trips at a time breaks things #145

Closed remulasce closed 9 years ago

remulasce commented 9 years ago

Trip to dismiss index is not properly preserved by the library, causing occasional crashes, and also messing up internal dismissals and scoping.

Fixed the crash with a try{} block, but it still enters and indeterminate state.

remulasce commented 9 years ago

OK, pausing the listener until the entire animation is done seems to prevent it. Need to watch out for the listener staying paused for too long.

remulasce commented 9 years ago

Testing shows that doesn't happen. Closing.