Anonymous | Login | Signup for a new account | 21-11-24 12:39 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 | ||||
0001498 | SAS.Планета | Рефакторинг | 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 | ||||||
Platform | OS | OS Version | |||||||
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. | ||||||||
Attached Files | |||||||||
Notes | |
(0008428) vasketsov (manager) 13-08-2012 18:42 |
Смогу начать не раньше начала сентября. >при этом вызываются функции WinApi c суфиксом A Имено по сабжевой причине и сразу сделано с суффиксом. |
(0008430) vasketsov (manager) 13-08-2012 21:58 |
>Скорее всего менять везде String на AnsiString и PChar на PAnsiChar Поназаменял везде у себя (а также Char на AnsiChar), кроме тем мест, где явно: а) возможно указание String (отметил такие места указанием // String OK); б) требуется WideString (в частности парсер XML). По идее должно работать. Я вроде всё что можно проверил, косяков не нашёл. Сам смогу проверить на XE2 видимо не раньше сентября, так как сваливаю до конца месяца и буду недоступен. Потом будем потихоньку назад превращать Ansi в Wide местами, если где потребуется. |
(0008439) vdemidov (manager) 14-08-2012 07:55 |
Дай мне права на vsagps. Там еще пару мест осталось, которые нужно подправить. |
(0008440) vasketsov (manager) 14-08-2012 08:00 edited on: 14-08-2012 08:06 |
А у тя разве их нет? Ну дам конечно. Пара мест? Я поиском искал по PChar, целым словам Char и String - хз где ты там ещё чего нашёл. Чё-то Access management глючит, список юзеров недоступен (( Убегать надо, так что вечером только смогу. Хотя про пару мест мог бы и тут написать. |
(0008441) vdemidov (manager) 14-08-2012 08:07 |
Ну например сравнение с DecimalSeparator который именно Char |
(0008442) vdemidov (manager) 14-08-2012 08:08 |
Еще есть в vsagps_runtime строчка: dwErr:=StrLen(@(theDevDetailData^.DevicePath)); которую нужно поменять на dwErr:=StrLen(PAnsiChar(@(theDevDetailData^.DevicePath))); |
(0008443) vasketsov (manager) 14-08-2012 08:10 edited on: 14-08-2012 08:19 |
В vsagps_public_print надо везде сделать без Ansi просто (перестаралсо). зы. нет, наврал, оно же для записи лога тоже юзается (( думать надо )) ззы. вроде бы DevicePath и так AnsiChar - указатель на него не PAnsiChar? ну да бог с ним, поправил уже. |
(0008481) vasketsov (manager) 14-08-2012 11:48 |
>сравнение с DecimalSeparator С этим сегодня поборюсь. Ещё нашлось чего? Всё ещё нужен доступ? Или собралось и работает? А то у меня пров видимо шалит, bb еле шевелится, пока никак, и тут топики секундами открываются. |
(0008483) vdemidov (manager) 14-08-2012 12:03 |
Лучше дай. Там осталась огромная туча варнингов на тему преобразований в wideString и обратно. |
(0008485) vasketsov (manager) 14-08-2012 12:07 |
ОК. В личку отпишусь. |
Issue History | |||
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 | View Revisions |
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 | View Revisions |
14-08-2012 08:19 | vasketsov | Note Edited: 0008443 | View Revisions |
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 |
My View | View Issues | Change Log | Roadmap | Search |
Copyright © 2007 - 2024 SAS.Planet Team |