SASGIS - SAS.Планета
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001498SAS.ПланетаРефакторингpublic13-08-2012 18:3610-10-2012 14:16
Reportervdemidov 
Assigned Tovasketsov 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version120808 
Target Version120808Fixed in Version120808 
Summary0001498: Подготовить vsagps для юникодных версий Delphi
DescriptionСейчас в vsagps дикая мешанина String, AnsiChar, PChar, PAnsiChar и при этом вызываются функции WinApi c суфиксом A
Нужно в этом наводить порядок. Скорее всего менять везде String на AnsiString и PChar на PAnsiChar. Но нужно очень внимательно смотреть.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0002053resolved vasketsov SACS.Планета Поддержка XE2 внутри VSAGPS 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
13-08-2012 18:36vdemidovNew Issue
13-08-2012 18:36vdemidovStatusnew => assigned
13-08-2012 18:36vdemidovAssigned To => vasketsov
13-08-2012 18:42vasketsovNote Added: 0008428
13-08-2012 21:58vasketsovNote Added: 0008430
13-08-2012 22:19vasketsovStatusassigned => resolved
13-08-2012 22:19vasketsovFixed in Version => 120808
13-08-2012 22:19vasketsovResolutionopen => fixed
14-08-2012 07:55vdemidovNote Added: 0008439
14-08-2012 08:00vasketsovNote Added: 0008440
14-08-2012 08:06vasketsovNote Edited: 0008440bug_revision_view_page.php?bugnote_id=8440#r4100
14-08-2012 08:07vdemidovNote Added: 0008441
14-08-2012 08:08vdemidovNote Added: 0008442
14-08-2012 08:10vasketsovNote Added: 0008443
14-08-2012 08:13vasketsovNote Edited: 0008443bug_revision_view_page.php?bugnote_id=8443#r4102
14-08-2012 08:19vasketsovNote Edited: 0008443bug_revision_view_page.php?bugnote_id=8443#r4103
14-08-2012 11:48vasketsovNote Added: 0008481
14-08-2012 12:03vdemidovNote Added: 0008483
14-08-2012 12:07vasketsovNote Added: 0008485
10-10-2012 11:40TolikStatusresolved => closed
10-10-2012 14:16vdemidovTarget Version121010 => 120808
30-07-2013 08:10vdemidovRelationship addedrelated to 0002053

Notes
(0008428)
vasketsov   
13-08-2012 18:42   
Смогу начать не раньше начала сентября.

>при этом вызываются функции WinApi c суфиксом A
Имено по сабжевой причине и сразу сделано с суффиксом.
(0008430)
vasketsov   
13-08-2012 21:58   
>Скорее всего менять везде String на AnsiString и PChar на PAnsiChar
Поназаменял везде у себя (а также Char на AnsiChar), кроме тем мест, где явно:
а) возможно указание String (отметил такие места указанием // String OK);
б) требуется WideString (в частности парсер XML).

По идее должно работать. Я вроде всё что можно проверил, косяков не нашёл. Сам смогу проверить на XE2 видимо не раньше сентября, так как сваливаю до конца месяца и буду недоступен.

Потом будем потихоньку назад превращать Ansi в Wide местами, если где потребуется.
(0008439)
vdemidov   
14-08-2012 07:55   
Дай мне права на vsagps. Там еще пару мест осталось, которые нужно подправить.
(0008440)
vasketsov   
14-08-2012 08:00   
(edited on: 14-08-2012 08:06)
А у тя разве их нет? Ну дам конечно.

Пара мест? Я поиском искал по PChar, целым словам Char и String - хз где ты там ещё чего нашёл.

Чё-то Access management глючит, список юзеров недоступен ((
Убегать надо, так что вечером только смогу.
Хотя про пару мест мог бы и тут написать.
(0008441)
vdemidov   
14-08-2012 08:07   
Ну например сравнение с DecimalSeparator который именно Char
(0008442)
vdemidov   
14-08-2012 08:08   
Еще есть в vsagps_runtime строчка:
dwErr:=StrLen(@(theDevDetailData^.DevicePath));
которую нужно поменять на
dwErr:=StrLen(PAnsiChar(@(theDevDetailData^.DevicePath)));
(0008443)
vasketsov   
14-08-2012 08:10   
(edited on: 14-08-2012 08:19)
В vsagps_public_print надо везде сделать без Ansi просто (перестаралсо).
зы. нет, наврал, оно же для записи лога тоже юзается ((
думать надо ))
ззы. вроде бы DevicePath и так AnsiChar - указатель на него не PAnsiChar? ну да бог с ним, поправил уже.
(0008481)
vasketsov   
14-08-2012 11:48   
>сравнение с DecimalSeparator
С этим сегодня поборюсь. Ещё нашлось чего? Всё ещё нужен доступ?

Или собралось и работает? А то у меня пров видимо шалит, bb еле шевелится, пока никак, и тут топики секундами открываются.
(0008483)
vdemidov   
14-08-2012 12:03   
Лучше дай. Там осталась огромная туча варнингов на тему преобразований в wideString и обратно.
(0008485)
vasketsov   
14-08-2012 12:07   
ОК. В личку отпишусь.