|
Notes |
|
|
(0013515)
|
|
zed
|
|
08-01-2014 10:35
|
|
|
Как полечить в общем-то очевидно, но не понятно почему этот интерфейс оказывается уже обнулённым. |
|
|
|
|
|
Потому что запрос закончился и объект уже начал удаляться. |
|
|
|
(0013521)
|
|
Garl
|
|
08-01-2014 12:21
|
|
|
много раз прилетал "превышен интервал ожидания" |
|
|
|
|
|
Ну вот похоже пару запросов еще висело, а завершение вышло не очень корректным. ИМХО там эту обработку вообще можно выкинуть, потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю. Нужно будет проверить. |
|
|
|
(0013523)
|
|
zed
|
|
08-01-2014 12:45
|
|
> потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю
Не путаешь, но даунлоадеры нужно немного дорабатывать. При загрузке тайлов нам нужно отслеживать 3 состояния:
- закрытие программы
- сдвиг карты или смена зума
- переключение на другую карту
Сейчас переключение карты не отслеживается (конкретно в даунлоадаре) и она не отваливается, хотя должна бы. Бывает включишь карту у которой сервер отвалился, переключишься на другую, а тебе ещё некоторое время будут прилетать ошибки "тайл не удалось загрузить" со старой карты. |
|
|
|
|
|
Ты путаешь загрузку видимой области и загрузку отдельного тайла. Я не вижу причины отменять запрос на загрузку этого отдельного тайла при сдвиге или смене карты. |
|
|
|
(0013525)
|
|
zed
|
|
08-01-2014 12:54
|
|
Все загрузки работают через одного даунлоадера и я не говорю, что при загрузке одного тайла нужны какие-то проверки. Я говорю, что саму качалку нужно исправлять и из-за этого может немного поменяться логика и при загрузке одного тайла. Т.е. может быть придётся создавать нотифаера, даже если он реально будет не нужен конкретно здесь.
Я к тому, что "ИМХО там эту обработку вообще можно выкинуть" (c) - пока не стоит спешить. |
|
|
|
|
|
При чем тут систему загрузки карты? Она может работать вообще по области для неактивной карты. Ты главное не путай загрузку видимой области, загрузку отдельных тайлов, загрузку по полигону и подсистему загрузки карты, которой все эти загрузки ставят задания. Так что эту обработку таки можно выкинуть. |
|
|
|
(0013527)
|
|
zed
|
|
08-01-2014 13:09
|
|
> Ты главное не путай
Ничего я не путаю. И от FTileDownloadFinishListener вообще планирую избавиться. |
|
|
|
|
|
Та я понял. Но в свете конкретно этого бага, стоит вообще от отдельного потока TTileDownloaderUIOneTile отказаться. Постановка в очередь на закачку не сильно не заморозит интерфейс, а проверка наличия тайла и так выполняется в гуе. |
|
|
|
(0013530)
|
|
zed
|
|
08-01-2014 17:36
|
|
|
|
|
(0013531)
|
|
Garl
|
|
10-01-2014 04:40
|
|
|
пока полёт нормальный, но сильно не гонял. тикет пока не закрывайте |
|
|
|
(0013544)
|
|
Garl
|
|
12-01-2014 09:08
|
|
|
есть подозрение что нужно ещё отслеживать и изменение URL карты через настройки. |
|
|
|
(0013554)
|
|
zed
|
|
12-01-2014 16:47
|
|
|
В прикрепленном отчёте нету AV, там какая-то утечка и не факт, что она связана с закачкой. |
|
SASPlanet.Debug.elf (68,728) 07-01-2014 17:06