View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002308 | SAS.Планета | Баг / Bug | public | 07-01-2014 17:06 | 30-01-2014 13:02 |
| Reporter | Garl | Assigned To | zed | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | resolved | Resolution | fixed | ||
| Platform | Windows | OS | 7 | OS Version | Professional |
| Product Version | .Nightly | ||||
| Target Version | 140303 | Fixed in Version | 140303 | ||
| Summary | 0002308: Access violation at address 006C2A6E in module 'SASPlanet.Debug.exe'. Read of address 00000000 | ||||
| Description | Собственно вот такие штуки периодически вываливаются при закрытии программы | ||||
| Tags | No tags attached. | ||||
| Attached Files | |||||
|
|
Как полечить в общем-то очевидно, но не понятно почему этот интерфейс оказывается уже обнулённым. |
|
|
Потому что запрос закончился и объект уже начал удаляться. |
|
|
много раз прилетал "превышен интервал ожидания" |
|
|
Ну вот похоже пару запросов еще висело, а завершение вышло не очень корректным. ИМХО там эту обработку вообще можно выкинуть, потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю. Нужно будет проверить. |
|
|
> потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю Не путаешь, но даунлоадеры нужно немного дорабатывать. При загрузке тайлов нам нужно отслеживать 3 состояния: - закрытие программы - сдвиг карты или смена зума - переключение на другую карту Сейчас переключение карты не отслеживается (конкретно в даунлоадаре) и она не отваливается, хотя должна бы. Бывает включишь карту у которой сервер отвалился, переключишься на другую, а тебе ещё некоторое время будут прилетать ошибки "тайл не удалось загрузить" со старой карты. |
|
|
Ты путаешь загрузку видимой области и загрузку отдельного тайла. Я не вижу причины отменять запрос на загрузку этого отдельного тайла при сдвиге или смене карты. |
|
|
Все загрузки работают через одного даунлоадера и я не говорю, что при загрузке одного тайла нужны какие-то проверки. Я говорю, что саму качалку нужно исправлять и из-за этого может немного поменяться логика и при загрузке одного тайла. Т.е. может быть придётся создавать нотифаера, даже если он реально будет не нужен конкретно здесь. Я к тому, что "ИМХО там эту обработку вообще можно выкинуть" (c) - пока не стоит спешить. |
|
|
При чем тут систему загрузки карты? Она может работать вообще по области для неактивной карты. Ты главное не путай загрузку видимой области, загрузку отдельных тайлов, загрузку по полигону и подсистему загрузки карты, которой все эти загрузки ставят задания. Так что эту обработку таки можно выкинуть. |
|
|
> Ты главное не путай Ничего я не путаю. И от FTileDownloadFinishListener вообще планирую избавиться. |
|
|
Та я понял. Но в свете конкретно этого бага, стоит вообще от отдельного потока TTileDownloaderUIOneTile отказаться. Постановка в очередь на закачку не сильно не заморозит интерфейс, а проверка наличия тайла и так выполняется в гуе. |
|
|
Garl, проверяй завтра. |
|
|
пока полёт нормальный, но сильно не гонял. тикет пока не закрывайте |
|
|
есть подозрение что нужно ещё отслеживать и изменение URL карты через настройки. |
|
|
В прикрепленном отчёте нету AV, там какая-то утечка и не факт, что она связана с закачкой. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 07-01-2014 17:06 | Garl | New Issue | |
| 07-01-2014 17:06 | Garl | File Added: SASPlanet.Debug.elf | |
| 08-01-2014 10:35 | zed | Note Added: 0013515 | |
| 08-01-2014 10:57 | vdemidov | Note Added: 0013518 | |
| 08-01-2014 12:21 | Garl | Note Added: 0013521 | |
| 08-01-2014 12:23 | vdemidov | Note Added: 0013522 | |
| 08-01-2014 12:45 | zed | Note Added: 0013523 | |
| 08-01-2014 12:50 | vdemidov | Note Added: 0013524 | |
| 08-01-2014 12:54 | zed | Note Added: 0013525 | |
| 08-01-2014 13:02 | vdemidov | Note Added: 0013526 | |
| 08-01-2014 13:09 | zed | Note Added: 0013527 | |
| 08-01-2014 13:38 | vdemidov | Note Added: 0013528 | |
| 08-01-2014 17:36 | zed | Note Added: 0013530 | |
| 08-01-2014 17:36 | zed | Assigned To | => zed |
| 08-01-2014 17:36 | zed | Status | new => feedback |
| 10-01-2014 04:40 | Garl | Note Added: 0013531 | |
| 10-01-2014 04:40 | Garl | Status | feedback => assigned |
| 10-01-2014 04:40 | Garl | Status | assigned => feedback |
| 12-01-2014 08:55 | Garl | File Added: 12-01-SASPlanet.Debug.elf | |
| 12-01-2014 09:08 | Garl | Note Added: 0013544 | |
| 12-01-2014 09:08 | Garl | Status | feedback => assigned |
| 12-01-2014 16:47 | zed | Note Added: 0013554 | |
| 30-01-2014 11:03 | vdemidov | Target Version | => 140303 |
| 30-01-2014 13:02 | zed | Status | assigned => resolved |
| 30-01-2014 13:02 | zed | Fixed in Version | => 140303 |
| 30-01-2014 13:02 | zed | Resolution | open => fixed |
| 08-08-2025 13:22 | zed | Category | Баг => Баг / Bug |