View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001684 | Far Manager | Panel | public | 2011-01-20 11:08 | 2011-12-23 08:53 |
Reporter | Maximus | Assigned To | vskirdin | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | x86 | OS | Windows | OS Version | 7 |
Product Version | 2.0 | ||||
Fixed in Version | 2.0 | ||||
Summary | 0001684: Падение FileList:GetCurName во время чтения сетевой папки | ||||
Description | Падает из-за того, что из CheckForEscSilent вызывается ProcessSynchroEvent. А плагин зовет ACTL_GETWINDOWINFO. Фар еще не закончил чтение каталога и CurFile больше чем FileCount. Что приводит к падению. Наверное нужно запретить синхро из CheckForEscSilent? Да и assert-ов в filelist.cpp повтыкать не помешало бы, чтобы видеть проблему сразу, а не тогда, когда случайно на строковой функции свалится | ||||
Tags | No tags attached. | ||||
Build | 1804 | ||||
|
|
|
как-то так, например? |
|
а зачем лишнее условие? if (PrevMouseButtonState!=MouseButtonState) PrevMouseButtonState=MouseButtonState; |
|
|
|
отлаживался, мешалось - брекпойнт по значению всплывал постоянно убрал условие, поправил один ассерт в конце |
|
в FileList::GetSelCount() спорный момент на счет проверки в ассерте "!FileCount". Может имеет смысл уточнить assert(!(CurFile>=FileCount && FileCount && (ReturnCurrentFile||!SelFileCount))); ? |
|
наверное так: assert(!FileCount || !(ReturnCurrentFile||!SelFileCount) || (CurFile<FileCount)); |
|
после build 1804 |
Date Modified | Username | Field | Change |
---|---|---|---|
2011-01-20 11:08 | Maximus | New Issue | |
2011-01-20 21:19 | Maximus | File Added: 110120.diff.7z | |
2011-01-20 21:19 | Maximus | Note Added: 0006227 | |
2011-01-28 06:23 | vskirdin | Note Added: 0006264 | |
2011-01-28 19:59 | Maximus | File Added: 110128.diff.7z | |
2011-01-28 20:00 | Maximus | Note Added: 0006267 | |
2011-01-29 06:02 | vskirdin | Note Added: 0006268 | |
2011-01-30 15:07 | Maximus | Note Added: 0006285 | |
2011-01-31 11:19 | vskirdin | Assigned To | => vskirdin |
2011-01-31 11:19 | vskirdin | Status | new => assigned |
2011-01-31 11:20 | vskirdin | Note Added: 0006293 | |
2011-01-31 11:20 | vskirdin | Status | assigned => feedback |
2011-12-23 08:53 | alexy | Build | => 1804 |
2011-12-23 08:53 | alexy | Status | feedback => closed |
2011-12-23 08:53 | alexy | Resolution | open => fixed |
2011-12-23 08:53 | alexy | Fixed in Version | => 2.0 |