View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000510 | Far Manager 1.7x | Dialog | public | 2008-03-25 08:46 | 2008-03-30 01:01 |
| Reporter | NightRoman | Assigned To | DrKnS | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | x86 | OS | Windows | OS Version | XP SP1 |
| Product Version | 1.71 | ||||
| Fixed in Version | 1.71 | ||||
| Summary | 0000510: Тип FarDialogEvent.Result должен быть PLONG_PTR, а не LONG_PTR | ||||
| Description | Сейчас тип FarDialogEvent.Result - LONG_PTR, неудобно, кастинги надо делать непонятно зачем, а должен быть PLONG_PTR, потому что это указатель, по определению. Это не breaking change, ни у кого вроде ничего не должно отвалиться ни бинарно (sizeof тот же), ни при компиляции, в худшем случае, будет приведение PLONG_PTR к нему же - масло масляное, но это не должно быть ошибкой, на первый взгляд. | ||||
| Tags | No tags attached. | ||||
| Build | 2346 | ||||
|
|
Тут надо подумать, я что то вообще не понимаю почему Result это указатель а не то что вернул/возвратит обработчик. Надо как я думаю переделать пока не поздно и тогда PLONG_PTR будет не причём. |
|
|
Хм... Действительно. Тогда, пока никто не использовал это в плагинах, делаем Result просто возвращаемым значением. После 2346/462. Писание поправлено. |
|
|
я уже заюзал, кой у кого что-то может отвалится (не у многих). но согласен, пока не очень поздно, лучше переделать. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2008-03-25 08:46 | NightRoman | New Issue | |
| 2008-03-25 17:01 | alexy | Note Added: 0001429 | |
| 2008-03-25 17:01 | alexy | Status | new => assigned |
| 2008-03-25 17:01 | alexy | Assigned To | => DrKnS |
| 2008-03-26 02:36 | DrKnS | Note Added: 0001431 | |
| 2008-03-26 02:36 | DrKnS | Status | assigned => feedback |
| 2008-03-26 04:55 | NightRoman | Note Added: 0001432 | |
| 2008-03-30 01:01 | DrKnS | Build | => 2346 |
| 2008-03-30 01:01 | DrKnS | Status | feedback => closed |
| 2008-03-30 01:01 | DrKnS | Resolution | open => fixed |
| 2008-03-30 01:01 | DrKnS | Fixed in Version | => 1.71 |