View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001498 | SAS.Планета | Рефакторинг / Refactoring | public | 13-08-2012 18:36 | 10-10-2012 14:16 |
| Reporter | vdemidov | Assigned To | vasketsov | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | 120808 | ||||
| Target Version | 120808 | Fixed in Version | 120808 | ||
| Summary | 0001498: Подготовить vsagps для юникодных версий Delphi | ||||
| Description | Сейчас в vsagps дикая мешанина String, AnsiChar, PChar, PAnsiChar и при этом вызываются функции WinApi c суфиксом A Нужно в этом наводить порядок. Скорее всего менять везде String на AnsiString и PChar на PAnsiChar. Но нужно очень внимательно смотреть. | ||||
| Tags | No tags attached. | ||||
|
|
Смогу начать не раньше начала сентября. >при этом вызываются функции WinApi c суфиксом A Имено по сабжевой причине и сразу сделано с суффиксом. |
|
|
>Скорее всего менять везде String на AnsiString и PChar на PAnsiChar Поназаменял везде у себя (а также Char на AnsiChar), кроме тем мест, где явно: а) возможно указание String (отметил такие места указанием // String OK); б) требуется WideString (в частности парсер XML). По идее должно работать. Я вроде всё что можно проверил, косяков не нашёл. Сам смогу проверить на XE2 видимо не раньше сентября, так как сваливаю до конца месяца и буду недоступен. Потом будем потихоньку назад превращать Ansi в Wide местами, если где потребуется. |
|
|
Дай мне права на vsagps. Там еще пару мест осталось, которые нужно подправить. |
|
|
А у тя разве их нет? Ну дам конечно. Пара мест? Я поиском искал по PChar, целым словам Char и String - хз где ты там ещё чего нашёл. Чё-то Access management глючит, список юзеров недоступен (( Убегать надо, так что вечером только смогу. Хотя про пару мест мог бы и тут написать. |
|
|
Ну например сравнение с DecimalSeparator который именно Char |
|
|
Еще есть в vsagps_runtime строчка: dwErr:=StrLen(@(theDevDetailData^.DevicePath)); которую нужно поменять на dwErr:=StrLen(PAnsiChar(@(theDevDetailData^.DevicePath))); |
|
|
В vsagps_public_print надо везде сделать без Ansi просто (перестаралсо). зы. нет, наврал, оно же для записи лога тоже юзается (( думать надо )) ззы. вроде бы DevicePath и так AnsiChar - указатель на него не PAnsiChar? ну да бог с ним, поправил уже. |
|
|
>сравнение с DecimalSeparator С этим сегодня поборюсь. Ещё нашлось чего? Всё ещё нужен доступ? Или собралось и работает? А то у меня пров видимо шалит, bb еле шевелится, пока никак, и тут топики секундами открываются. |
|
|
Лучше дай. Там осталась огромная туча варнингов на тему преобразований в wideString и обратно. |
|
|
ОК. В личку отпишусь. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 13-08-2012 18:36 | vdemidov | New Issue | |
| 13-08-2012 18:36 | vdemidov | Status | new => assigned |
| 13-08-2012 18:36 | vdemidov | Assigned To | => vasketsov |
| 13-08-2012 18:42 | vasketsov | Note Added: 0008428 | |
| 13-08-2012 21:58 | vasketsov | Note Added: 0008430 | |
| 13-08-2012 22:19 | vasketsov | Status | assigned => resolved |
| 13-08-2012 22:19 | vasketsov | Fixed in Version | => 120808 |
| 13-08-2012 22:19 | vasketsov | Resolution | open => fixed |
| 14-08-2012 07:55 | vdemidov | Note Added: 0008439 | |
| 14-08-2012 08:00 | vasketsov | Note Added: 0008440 | |
| 14-08-2012 08:06 | vasketsov | Note Edited: 0008440 | |
| 14-08-2012 08:07 | vdemidov | Note Added: 0008441 | |
| 14-08-2012 08:08 | vdemidov | Note Added: 0008442 | |
| 14-08-2012 08:10 | vasketsov | Note Added: 0008443 | |
| 14-08-2012 08:13 | vasketsov | Note Edited: 0008443 | |
| 14-08-2012 08:19 | vasketsov | Note Edited: 0008443 | |
| 14-08-2012 11:48 | vasketsov | Note Added: 0008481 | |
| 14-08-2012 12:03 | vdemidov | Note Added: 0008483 | |
| 14-08-2012 12:07 | vasketsov | Note Added: 0008485 | |
| 10-10-2012 11:40 | Tolik | Status | resolved => closed |
| 10-10-2012 14:16 | vdemidov | Target Version | 121010 => 120808 |
| 30-07-2013 08:10 | vdemidov | Relationship added | related to 0002053 |
| 08-08-2025 13:25 | zed | Category | Рефакторинг => Рефакторинг / Refactoring |