Anonymous | Login | Signup for a new account | 21-11-24 18:01 UTC |
All Projects | SAS.Планета | Домен, сайт, форум, багтрекер | Доработка карты (ZMP) | Переводы и локализации | Прочее |
My View | View Issues | Change Log | Roadmap | Search |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0002308 | SAS.Планета | [All Projects] Баг | 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 | SASPlanet.Debug.elf [^] (68,728 bytes) 07-01-2014 17:06 12-01-SASPlanet.Debug.elf [^] (177,228 bytes) 12-01-2014 08:55 | ||||||||
Notes | |
(0013515) zed (manager) 08-01-2014 10:35 |
Как полечить в общем-то очевидно, но не понятно почему этот интерфейс оказывается уже обнулённым. |
(0013518) vdemidov (manager) 08-01-2014 10:57 |
Потому что запрос закончился и объект уже начал удаляться. |
(0013521) Garl (manager) 08-01-2014 12:21 |
много раз прилетал "превышен интервал ожидания" |
(0013522) vdemidov (manager) 08-01-2014 12:23 |
Ну вот похоже пару запросов еще висело, а завершение вышло не очень корректным. ИМХО там эту обработку вообще можно выкинуть, потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю. Нужно будет проверить. |
(0013523) zed (manager) 08-01-2014 12:45 |
> потому что все даунлоадеры уведомляются о завершении приложения напрямую, если я ничего не путаю Не путаешь, но даунлоадеры нужно немного дорабатывать. При загрузке тайлов нам нужно отслеживать 3 состояния: - закрытие программы - сдвиг карты или смена зума - переключение на другую карту Сейчас переключение карты не отслеживается (конкретно в даунлоадаре) и она не отваливается, хотя должна бы. Бывает включишь карту у которой сервер отвалился, переключишься на другую, а тебе ещё некоторое время будут прилетать ошибки "тайл не удалось загрузить" со старой карты. |
(0013524) vdemidov (manager) 08-01-2014 12:50 |
Ты путаешь загрузку видимой области и загрузку отдельного тайла. Я не вижу причины отменять запрос на загрузку этого отдельного тайла при сдвиге или смене карты. |
(0013525) zed (manager) 08-01-2014 12:54 |
Все загрузки работают через одного даунлоадера и я не говорю, что при загрузке одного тайла нужны какие-то проверки. Я говорю, что саму качалку нужно исправлять и из-за этого может немного поменяться логика и при загрузке одного тайла. Т.е. может быть придётся создавать нотифаера, даже если он реально будет не нужен конкретно здесь. Я к тому, что "ИМХО там эту обработку вообще можно выкинуть" (c) - пока не стоит спешить. |
(0013526) vdemidov (manager) 08-01-2014 13:02 |
При чем тут систему загрузки карты? Она может работать вообще по области для неактивной карты. Ты главное не путай загрузку видимой области, загрузку отдельных тайлов, загрузку по полигону и подсистему загрузки карты, которой все эти загрузки ставят задания. Так что эту обработку таки можно выкинуть. |
(0013527) zed (manager) 08-01-2014 13:09 |
> Ты главное не путай Ничего я не путаю. И от FTileDownloadFinishListener вообще планирую избавиться. |
(0013528) vdemidov (manager) 08-01-2014 13:38 |
Та я понял. Но в свете конкретно этого бага, стоит вообще от отдельного потока TTileDownloaderUIOneTile отказаться. Постановка в очередь на закачку не сильно не заморозит интерфейс, а проверка наличия тайла и так выполняется в гуе. |
(0013530) zed (manager) 08-01-2014 17:36 |
Garl, проверяй завтра. |
(0013531) Garl (manager) 10-01-2014 04:40 |
пока полёт нормальный, но сильно не гонял. тикет пока не закрывайте |
(0013544) Garl (manager) 12-01-2014 09:08 |
есть подозрение что нужно ещё отслеживать и изменение URL карты через настройки. |
(0013554) zed (manager) 12-01-2014 16:47 |
В прикрепленном отчёте нету AV, там какая-то утечка и не факт, что она связана с закачкой. |
Issue History | |||
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 |
My View | View Issues | Change Log | Roadmap | Search |
Copyright © 2007 - 2024 SAS.Planet Team |