Ros-diffs August 2018

ros-diffs@reactos.org

32 participants
304 discussions

01/01: [SDK][Translation] Polish translation update (#495)

by Paweł Cholewa

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8569c905ca2465e1aa120… commit 8569c905ca2465e1aa1208fd5976de7ef198bf54 Author: Paweł Cholewa <MCPlayG(a)wp.pl> AuthorDate: Sun Aug 19 19:47:26 2018 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sun Aug 19 19:47:26 2018 +0200 [SDK][Translation] Polish translation update (#495) * Polish translation update in neteventmsg.mc --- sdk/include/reactos/mc/neteventmsg.mc | 208 +++++++++++++++++++++++++++++++++- 1 file changed, 205 insertions(+), 3 deletions(-) diff --git a/sdk/include/reactos/mc/neteventmsg.mc b/sdk/include/reactos/mc/neteventmsg.mc index 5a31bcda75..220a24c6f5 100644 --- a/sdk/include/reactos/mc/neteventmsg.mc +++ b/sdk/include/reactos/mc/neteventmsg.mc @@ -22,8 +22,8 @@ FacilityNames=(System=0x0:FACILITY_SYSTEM LanguageNames=(English=0x409:MSG00409 Russian=0x419:MSG00419 - French=0x040c:MSG0040c) - + French=0x040c:MSG0040c + Polish=0x415:MSG00415) ; ; message definitions @@ -48,6 +48,9 @@ Language=Russian Language=French Le fichier de journalisation %1 est plein. . +Language=Polish +Plik dziennika %1 jest zapełniony. +. MessageId=6001 Severity=Warning @@ -62,6 +65,9 @@ Language=Russian Language=French Le fichier de journalisation %1 ne peut pas être ouvert. . +Language=Polish +Nie można otworzyć pliku dziennika %1. +. MessageId=6002 Severity=Warning @@ -76,6 +82,9 @@ Language=Russian Language=French Le fichier de journalisation %1 est corrompu et doit être vidé. . +Language=Polish +Plik dziennika "%1" jest uszkodzony i zostanie wyczyszczony. +. MessageId=6003 Severity=Warning @@ -90,6 +99,9 @@ Language=Russian Language=French Le fichier de journalisation des application ne peut pas être ouvert. %1 sera utilisé comme fichier de journalisation par défaut. . +Language=Polish +Nie można otworzyć pliku dziennika aplikacji. Jako domyślny plik dziennika zostanie użyty plik %1. +. MessageId=6004 Severity=Warning @@ -104,6 +116,9 @@ Language=Russian Language=French Un paquet de pilote reçu par le sous-système IO était invalide. La donnée est le paquet. . +Language=Polish +Pakiet sterownika otrzymany z podsystemu wejścia/wyjścia jest niepoprawny. Przedstawione dane to otrzymany pakiet. +. MessageId=6005 Severity=Warning @@ -118,6 +133,9 @@ Language=Russian Language=French Le service de journalisation des évènements a été démarré. . +Language=Polish +Uruchomiono usługę Dziennik zdarzeń. +. MessageId=6006 Severity=Warning @@ -132,6 +150,9 @@ Language=Russian Language=French Le service de journalisation des évènements a été arrêté. . +Language=Polish +Zatrzymano usługę Dziennik zdarzeń. +. MessageId=6007 Severity=Warning @@ -155,6 +176,9 @@ Language=Russian Language=French Service de journalisation des évènements %0 . +Language=Polish +Usługa Dziennik zdarzeń "%0" +. MessageId=6008 Severity=Warning @@ -169,6 +193,9 @@ Language=Russian Language=French L'arrêt précédent du système à %1 le %2 n'était pas prévu. . +Language=Polish +Poprzednie zamknięcie systemu o godzinie %1 na %2 było nieoczekiwane. +. MessageId=6009 Severity=Warning @@ -197,6 +224,9 @@ Language=Russian Language=French Le service %1 n'a pas réussi à installer une source d'évènement. . +Language=Polish +Usługa %1 nie mogła skonfigurować źródła zdarzenia. +. MessageId=6011 Severity=Error @@ -211,6 +241,9 @@ NetBIOS и DNS имена этого компьютера были измене Language=French Le nom NetBIOS et le nom d'hôte DNS de cette machine ont été changés de %1 à %2. . +Language=Polish +Nazwa NetBIOS i nazwa hosta DNS tego komputera została zmieniona z %1 na %2. +. MessageId=6012 Severity=Error @@ -225,6 +258,9 @@ DNS-домен присвоенный этому компьютеру был и Language=French Le domaine DNS assigné à cet ordinateur a été changé de %1 à %2. . +Language=Polish +Domena DNS przydzielona do tego komputera uległa zmianie z %1 na %2. +. ; @@ -244,6 +280,9 @@ Language=Russian Language=French Un pilote spécifique monoprocesseur a été chargé sur un système multiprocesseurs. Le pilote n'a pas pu être chargé. . +Language=Polish +Próbowano załadować sterownik jednoprocesorowy na komputerze wieloprocesorowym. Nie można załadować sterownika. +. ; @@ -263,6 +302,9 @@ Language=Russian Language=French Le service %1 n'a pas pu démarrer en raison de l'erreur suivante : %n%2 . +Language=Polish +Nie można uruchomić usługi %1 z powodu następującego błędu: %n%2 +. MessageId=7001 Severity=Error @@ -277,6 +319,9 @@ Language=Russian Language=French Le serveur %1 dépend du service %2 qui n'a pas pu démarrer en raison de l'erreur suivante : %n%3 . +Language=Polish +Usługa %1 zależy od usługi %2, której nie można uruchomić z powodu następującego błędu: %n%3 +. MessageId=7002 Severity=Error @@ -291,6 +336,9 @@ Language=Russian Language=French Le service %1 dépend du groupe %2 et aucun membre de ce groupe n'a démarré. . +Language=Polish +Usługa %1 zależy od grupy %2, a nie uruchomiono żadnego członka tej grupy. +. MessageId=7003 Severity=Error @@ -305,6 +353,9 @@ Language=Russian Language=French Le service %1 dépend du service non existant suivant : %2 . +Language=Polish +Usługa %1 zależy od następującej nieistniejącej usługi: %2 +. MessageId=7005 Severity=Error @@ -319,6 +370,9 @@ Language=Russian Language=French L'appel %1 a échoué avec l'erreur suivante : %n%2 . +Language=Polish +Wywołanie %1 nie powiodło się i wystąpił następujący błąd: %n%2 +. MessageId=7006 Severity=Error @@ -333,6 +387,9 @@ The %1 call failed for %2 with the following error: %n%3 Language=French L'appel %1 a échoué pour %2 avec l'erreur suivante : %n%3 . +Language=Polish +Wywołanie %1 dla %2 nie powiodło się i wystąpił następujący błąd: %n%3 +. MessageId=7007 Severity=Error @@ -347,6 +404,9 @@ The system reverted to its last known good configuration. The system is restart Language=French Le système a restauré sa dernière bonne configuration connue. Le système redémarre... . +Language=Polish +System powrócił do ostatniej znanej dobrej konfiguracji. Trwa ponowne uruchamianie systemu... +. MessageId=7008 Severity=Error @@ -361,6 +421,9 @@ No backslash is in the account name. Language=French Aucun backslash n'est présent dans le nom de compte. . +Language=Polish +W nazwie konta brak znaku ukośnika odwrotnego. +. MessageId=7009 Severity=Error @@ -375,6 +438,9 @@ Timeout (%1 milliseconds) waiting for the %2 service to connect. Language=French Expiration du délai (%1 millisecondes) lors de l'attente du service %2 pour se connecter. . +Language=Polish +Upłynął limit czasu (%1 milisekund) podczas oczekiwania na połączenie się z usługą %2. +. MessageId=7010 Severity=Error @@ -389,6 +455,9 @@ Timeout (%1 milliseconds) waiting for ReadFile. Language=French Expiration du délai (%1 millisecondes) lors de l'attente de ReadFile. . +Language=Polish +Upłynął limit czasu (%1 milisekund) podczas oczekiwania na operację ReadFile. +. MessageId=7011 Severity=Error @@ -403,6 +472,9 @@ Timeout (%1 milliseconds) waiting for a transaction response from the %2 service Language=French Expiration du délai (%1 millisecondes) lors de l'attente pour une transaction réponse du service %2. . +Language=Polish +Upłynął limit czasu (%1 milisekund) podczas oczekiwania na odpowiedź transakcji z usługi %2. +. MessageId=7012 Severity=Error @@ -417,6 +489,9 @@ Message returned in transaction has incorrect size. Language=French Le message retourné dans la transaction a une taille incorrecte. . +Language=Polish +Wiadomość zwrócona w transakcji ma niepoprawny rozmiar. +. MessageId=7013 Severity=Error @@ -431,6 +506,9 @@ Logon attempt with current password failed with the following error: %n%1 Language=French La tentative de connexion avec le mot de passe actuel a échoué avec l'erreur suivante : %n%1 . +Language=Polish +Próba logowania za pomocą aktualnego hasła nie powiodła się; wystąpił następujący błąd: %n%1 +. MessageId=7014 Severity=Error @@ -445,6 +523,9 @@ Second logon attempt with old password also failed with the following error: %n% Language=French La seconde tentative de connexion avec l'ancien mot de passe a également échoué avec l'erreur suivante : %n%1 . +Language=Polish +Druga próba logowania za pomocą starego hasła także się nie powiodła; wystąpił następujący błąd: %n%1 +. MessageId=7015 Severity=Error @@ -459,6 +540,9 @@ Boot-start or system-start driver (%1) must not depend on a service. Language=French Un pilote au démarrage ou système (%1) ne doit pas dépendre d'un service. . +Language=Polish +Sterowniki startu rozruchowego oraz systemowego (%1) nie mogą zależeć od usługi. +. MessageId=7016 Severity=Error @@ -473,6 +557,9 @@ The %1 service has reported an invalid current state %2. Language=French Le service %1 a reporté un état courant invalide %2. . +Language=Polish +Usługa %1 zaraportowała nieprawidłowy stan bieżący %2. +. MessageId=7017 Severity=Error @@ -487,6 +574,9 @@ Detected circular dependencies demand starting %1. Language=French Dépendances circulaires détectées lors du démarrage de %1. . +Language=Polish +Wykryte zależności cykliczne wymagają uruchomienia %1. +. MessageId=7018 Severity=Error @@ -501,6 +591,9 @@ Detected circular dependencies auto-starting services. Language=French Dépendances circulaires détectées lors du démarrage automatique des services. . +Language=Polish +Wykryto automatycznie uruchamiane usługi zależności cyklicznych. +. MessageId=7019 Severity=Error @@ -515,6 +608,9 @@ Circular dependency: The %1 service depends on a service in a group which starts Language=French Dépendance circulaire : le service %1 dépend d'un service dans un groupe qui démarre plus tard. . +Language=Polish +Zależność cykliczna: usługa %1 zależy od usługi z grupy uruchamianej później. +. MessageId=7020 Severity=Error @@ -529,6 +625,9 @@ Circular dependency: The %1 service depends on a group which starts later. Language=French Dépendance circulaire : le service %1 dépend d'un groupe qui démarre plus tard. . +Language=Polish +Zależność cykliczna: usługa %1 zależy od grupy uruchamianej później. +. MessageId=7021 Severity=Error @@ -543,6 +642,9 @@ About to revert to the last known good configuration because the %1 service fail Language=French Sur le point de revenir à la dernière bonne configuration connue car le service %1 n'a pas réussi à démarrer. . +Language=Polish +System powróci do ostatniej znanej dobrej konfiguracji, ponieważ uruchomienie usługi %1 nie powiodło się. +. MessageId=7022 Severity=Error @@ -557,6 +659,9 @@ Language=Russian Language=French Le serveur %1 service s'est gelé au démarrage. . +Language=Polish +Usługa %1 zawiesiła się podczas uruchamiania. +. MessageId=7023 Severity=Error @@ -571,6 +676,9 @@ The %1 service terminated with the following error: %n%2 Language=French Le service %1 s'est terminé avec l'erreur suivante : %n%2 . +Language=Polish +Usługa %1 zakończyła działanie; wystąpił następujący błąd: %n%2 +. MessageId=7024 Severity=Error @@ -585,6 +693,9 @@ The %1 service terminated with service-specific error %2. Language=French Le service %1 s'est terminé avec une erreur spécifique au service %2. . +Language=Polish +Usługa %1 zakończyła działanie; wystąpił specyficzny dla niej błąd %2. +. MessageId=7025 Severity=Error @@ -599,6 +710,9 @@ At least one service or driver failed during system startup. Use Event Viewer t Language=French Au moins un service ou pilote a échoué durant le démarrage du système. Utilisez la visionneuse d'évènements pour examiner les journaux d'évènements pour plus de détails. . +Language=Polish +Przynajmniej jedna usługa lub jeden sterownik nie dały się uruchomić podczas uruchamiania systemu. Użyj Podglądu zdarzeń, aby znaleźć szczegółowe informacje w dzienniku zdarzeń. +. MessageId=7026 Severity=Error @@ -613,6 +727,9 @@ The following boot-start or system-start driver(s) failed to load: %1 Language=French Le pilote au démarrage ou système suivant n'a pas pu être chargé : %1 . +Language=Polish +Nie można załadować następujących sterowników startu rozruchowego lub systemowego: %1 +. MessageId=7027 Severity=Error @@ -627,6 +744,9 @@ ReactOS could not be started as configured. A previous working configuration wa Language=French ReactOS n'a pas pu démarrer tel que configuré. Une précédente configuration fonctionnelle a été utilisé à la place. . +Language=Polish +Nie można uruchomić systemu ReactOS zgodnie z aktualną konfiguracją. Zamiast niej użyto poprzedniej działającej konfiguracji. +. MessageId=7028 Severity=Error @@ -641,6 +761,9 @@ The %1 Registry key denied access to SYSTEM account programs so the Service Cont Language=French La clé de registre %1 a refusé l'accès aux programmes du compte SYSTEM. Le gestionnaire de contrôle des services s'est approprié la clé de registre. . +Language=Polish +Klucz Rejestru %1 nie przyznał dostępu do programów konta SYSTEM, w związku z czym Menedżer sterowania usługami przejął ten klucz na własność. +. MessageId=7029 Severity=Error @@ -655,6 +778,9 @@ Service Control Manager %0 Language=French Gestionnaire de contrôle des services %0 . +Language=Polish +Menedżer sterowania usługami %0 +. MessageId=7030 Severity=Error @@ -669,6 +795,9 @@ The %1 service is marked as an interactive service. However, the system is conf Language=French Le service %1 est marqué comme service interactif. Cependant, le système est configuré pour ne pas autoriser les services interactifs. Ce service pourrait ne pas fonctionner correctement. . +Language=Polish +Usługa %1 jest oznaczona jako usługa interakcyjna. System jest jednak skonfigurowany tak, aby nie zezwalać na usługi interakcyjne, dlatego ta usługa może nie działać właściwie. +. MessageId=7031 Severity=Error @@ -683,6 +812,9 @@ The %1 service terminated unexpectedly. It has done this %2 time(s). The follo Language=French Le service %1 s'est terminée de façon non prévue. Il l'a fait %2 fois. L'action corrective suivante va être prise dans %3 millisecondes : %5. . +Language=Polish +Usługa %1 niespodziewanie zakończyła pracę. Wystąpiło to razy: %2. W przeciągu %3 milisekund zostanie podjęta następująca czynność korekcyjna: %5. +. MessageId=7032 Severity=Error @@ -697,6 +829,9 @@ The Service Control Manager tried to take a corrective action (%2) after the une Language=French Le gestionnaire de contrôle des services a tenté de prendre une action corrective (%2) après la fin inattendue du service %3, mais cette action a échoué avec l'erreur suivante : %n%4 . +Language=Polish +Menedżer sterowania usługami próbował podjąć akcję korekcyjną (%2) po nieoczekiwanym zakończeniu usługi %3, ale ta akcja nie powiodła się przy następującym błędzie: %n%4. +. MessageId=7033 Severity=Error @@ -717,6 +852,10 @@ Le gestionnaire de contrôle des services ne s'est pas initialisé correctement. Le serveur de configuration de sécurité (scesrv.dll) n'a pas réussi à s'initialiser avec l'erreur %1. Le système redémarre... . +Language=Polish +Menedżer sterowania usługami nie został zainicjowany pomyślnie.\r\nNie udało się zainicjować serwera +konfiguracji zabezpieczeń (scesrv.dll)\r\nz powodu błędu %1. Trwa ponowne uruchamianie systemu... +. MessageId=7034 Severity=Error @@ -731,6 +870,9 @@ Language=Russian Language=French Le service %1 s'est terminée de façon non prévue. Il l'a fait %2 fois. . +Language=Polish +Usługa %1 niespodziewanie zakończyła pracę. Wystąpiło to razy: %2. +. MessageId=7035 Severity=Informational @@ -745,6 +887,9 @@ Language=Russian Language=French Le contrôle %2 a été envoyé avec succès au service %1. . +Language=Polish +Do usługi %1 został pomyślnie wysłany kod sterowania %2. +. MessageId=7036 Severity=Informational @@ -759,6 +904,9 @@ Language=Russian Language=French Le service %1 est entré dans l'état %2. . +Language=Polish +Usługa %1 weszła w stan %2. +. MessageId=7037 Severity=Error @@ -790,6 +938,15 @@ redémarrer le service %1 ou pourriez rencontrer d'autres erreurs. Pour vous ass que le service est configuré correctement, utilisez la console de gestion des services dans la console de gestion Microsoft (MMC). . +Language=Polish +Menedżer sterowania usługami napotkał błąd podczas cofania zmiany konfiguracji +usługi %1. Element usługi: %2 jest obecnie w stanie nieprzewidywalnym. + +Jeśli ta konfiguracja nie zostanie naprawiona, ponowne uruchomienie usługi +%1 może okazać się niemożliwe lub mogą wystąpić inne błędy. Aby upewnić się, +że usługa jest skonfigurowana właściwie, użyj przystawki Usługi w programie +ReactOS Management Console (MMC). +. MessageId=7038 Severity=Error @@ -813,6 +970,12 @@ actuellement configuré en raison de l'erreur suivante : %n%3%n%nPour vous assur que le service est correctement, utilisez la console de gestion des services dans la console de gestion Microsoft (MMC). . +Language=Polish +Usługa %1 nie mogła zalogować się jako %2 z aktualnie skonfigurowanym +hasłem z powodu następującego błędu: %n%3%n%nAby upewnić się, że usługa +jest skonfigurowana właściwie, użyj przystawki Usługi w programie +ReactOS Management Console (MMC). +. MessageId=7039 Severity=Warning @@ -839,6 +1002,13 @@ lancé le processus %2 et le processus %3 s'est connecté à la place.%n%n Veuillez noter que si ce service est configurer pour démarrer dans un débogueur, ce comportement est attendu. . +Language=Polish +Proces usługi innej niż uruchomiona przez Menedżer sterowania usługami +połączył się podczas uruchamiania usługi %1. Menedżer sterowania usługami uruchomił +proces %2, a połączył się proces %3.%n%n + +Jeśli ta usługa jest skonfigurowana do uruchomienia z debugerem, takie zachowanie jest oczekiwane. +. MessageId=7040 Severity=Informational @@ -853,6 +1023,9 @@ Language=Russian Language=French Le type de démarrage du service %1 a été changé de %2 à %3. . +Language=Polish +Typ startowy usługi %1 został zmieniony z %2 do %3. +. MessageId=7041 Severity=Error @@ -906,6 +1079,22 @@ Si vous avez déjà assigné ce droit d'utilisateur au compte de service, et que un objet de stratégie de groupe (GPO) associé avec ce nœud peut avoir supprimé le droit. Contactez votre administrateur de domaine pour vérifier si c'est ce qui se produit. . +Language=Polish +Usługa %1 nie mogła się zalogować jako %2 z aktualnie skonfigurowanym hasłem z powodu następującego błędu:%n +Błąd logowania: użytkownik nie otrzymał wymaganego typu logowania na tym komputerze.%n%n +Usługa: %1%n +Domena i użytkownik: %2%n%n +To konto usługi nie zawiera potrzebnego prawa użytkownika "Zaloguj się jako usługa."%n%n +Akcje użytkownika%n%n +Przypisz "Zaloguj się jako usługa" do konta usługi na tym komputerze. Możesz użyć +Ustawień zabezpieczeń lokalnych (Secpol.msc), by to wykonać. Jeśli ten komputer jest +węzłem w klastrze, sprawdź czy to prawo użytkownika jest przypisane do konta usługi +Klaster na wszystkich węzłach tego klastra.%n%n +Jeśli już przypisałeś to prawo użytkownika do konta usługi +i prawo użytkownika wygląda na usunięte, obiekt zasad grupy powiązany z tym +węzłem może usuwać to prawo. Skontaktuj się z administratorem domeny by sprawdzić, +czy to się dokonuje. +. MessageId=7042 Severity=Informational @@ -926,6 +1115,11 @@ Le contrôle %2 a été envoyé avec succès au service %1.%n%n La raison spécifiée était : %3 [%4]%n%n Commentaire : %5 . +Language=Polish +Do usługi %1 został pomyślnie wysłany kod sterowania %2.%n%n +Powód: %3 [%4]%n%n +Komentarz: %5 +. MessageId=7043 Severity=Error @@ -940,7 +1134,9 @@ The %1 service did not shut down properly after receiving a preshutdown control. Language=French Le service %1 ne s'est pas arrêté proprement après avoir reçu un contrôle de pré-arrêt. . - +Language=Polish +Usługa %1 nie została poprawnie wyłączona po otrzymaniu kodu sterownika przed zamknięciem systemu. +. ; ; transport events 9000-9499 @@ -959,6 +1155,9 @@ Language=Russian Language=French %2 n'a pas réussi à s'enregistrer avec le wrapper NDIS. . +Language=Polish +%2: nie można zarejestrować w otoce NDIS. +. MessageId=9006 Severity=Error @@ -973,3 +1172,6 @@ Language=Russian Language=French %2 n'a pas pu trouver l'adaptateur %3. . +Language=Polish +%2: nie można odnaleźć karty %3. +.

6 years, 4 months

01/01: [LOG2LINES] Enforce "Iso" type when unpacking the Iso file

by Serge Gautherie

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8283a9ece21c27a0edd43… commit 8283a9ece21c27a0edd438a6c3e5c482dd4f67aa Author: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> AuthorDate: Sun Jun 24 00:57:57 2018 +0200 Commit: Mark Jansen <mark.jansen(a)reactos.org> CommitDate: Sun Aug 19 17:39:15 2018 +0200 [LOG2LINES] Enforce "Iso" type when unpacking the Iso file Work around some "7z" dealing with files not named "*.iso" as "Cab" type. Issue at least with "7-Zip [64] 9.20" on "Ubuntu 16.04". CORE-14734 --- sdk/tools/log2lines/config.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/tools/log2lines/config.h b/sdk/tools/log2lines/config.h index 5f3d3cb6e9..e07dfa259b 100644 --- a/sdk/tools/log2lines/config.h +++ b/sdk/tools/log2lines/config.h @@ -21,7 +21,7 @@ #define CMD_7Z "7z" #define UNZIP_FMT_7Z "%s e -y %s -o%s > " DEV_NULL -#define UNZIP_FMT "%s x -y -r %s -o%s > " DEV_NULL +#define UNZIP_FMT "%s x -tIso -y -r %s -o%s > " DEV_NULL #define UNZIP_FMT_CAB \ "%s x -y -r %s" PATH_STR "reactos" PATH_STR "reactos.cab -o%s" \ PATH_STR "reactos" PATH_STR "reactos > " DEV_NULL

6 years, 4 months

02/02: [ATL_APITEST] Add test for CComQIPtr (or rather, our gcc hack that slightly looks like it)

by Mark Jansen

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4b8f6d5eaba7d55dfd2e3… commit 4b8f6d5eaba7d55dfd2e339d4d4564a62da1353b Author: Mark Jansen <mark.jansen(a)reactos.org> AuthorDate: Sun Aug 5 11:50:02 2018 +0200 Commit: Mark Jansen <mark.jansen(a)reactos.org> CommitDate: Sun Aug 19 17:35:18 2018 +0200 [ATL_APITEST] Add test for CComQIPtr (or rather, our gcc hack that slightly looks like it) --- modules/rostests/apitests/atl/CComQIPtr.cpp | 160 +++++++++++++++++++++ modules/rostests/apitests/atl/CMakeLists.txt | 1 + modules/rostests/apitests/atl/devenv/ATLTest.sln | 10 ++ .../apitests/atl/devenv/CComObject.vcxproj | 4 - .../{CSimpleMap.vcxproj => CComQIPtr.vcxproj} | 10 +- .../rostests/apitests/atl/devenv/CImage.vcxproj | 4 - .../apitests/atl/devenv/CSimpleArray.vcxproj | 4 - .../apitests/atl/devenv/CSimpleMap.vcxproj | 4 - .../rostests/apitests/atl/devenv/CString.vcxproj | 4 - modules/rostests/apitests/atl/testlist.c | 2 + 10 files changed, 176 insertions(+), 27 deletions(-) diff --git a/modules/rostests/apitests/atl/CComQIPtr.cpp b/modules/rostests/apitests/atl/CComQIPtr.cpp new file mode 100644 index 0000000000..f9fedb6898 --- /dev/null +++ b/modules/rostests/apitests/atl/CComQIPtr.cpp @@ -0,0 +1,160 @@ +/* + * PROJECT: ReactOS api tests + * LICENSE: GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+) + * PURPOSE: Test for CComQIPtr + * COPYRIGHT: Copyright 2018 Mark Jansen (mark.jansen(a)reactos.org) + */ + +#include <atlbase.h> +#include <atlcom.h> + +#ifdef __REACTOS__ + #include <apitest.h> +#else + #include <stdlib.h> + #include <stdio.h> + #include <stdarg.h> + int g_tests_executed = 0; + int g_tests_failed = 0; + void ok_func(const char *file, int line, BOOL value, const char *fmt, ...) + { + va_list va; + va_start(va, fmt); + if (!value) + { + printf("%s (%d): ", file, line); + vprintf(fmt, va); + g_tests_failed++; + } + g_tests_executed++; + va_end(va); + } + #undef ok + #define ok(value, ...) ok_func(__FILE__, __LINE__, value, __VA_ARGS__) + #define START_TEST(x) int main(void) +#endif + + +static LONG g_QI = 0; + +class CQITestObject : + public IPersist, + public IStdMarshalInfo +{ +public: + LONG m_dwRef; + + CQITestObject() + :m_dwRef(1) + { + } + ~CQITestObject() + { + } + + STDMETHOD_(ULONG, AddRef)() + { + InterlockedIncrement(&m_dwRef); + return 2; + } + + STDMETHOD_(ULONG, Release)() + { + InterlockedDecrement(&m_dwRef); + return 1; + } + + STDMETHOD(QueryInterface)(REFIID iid, void **ppvObject) + { + InterlockedIncrement(&g_QI); + if (iid == IID_IUnknown || iid == IID_IPersist) + { + AddRef(); + *ppvObject = static_cast<IPersist*>(this); + return S_OK; + } + else if (iid == IID_IStdMarshalInfo) + { + AddRef(); + *ppvObject = static_cast<IStdMarshalInfo*>(this); + return S_OK; + } + return E_NOINTERFACE; + } + + // *** IPersist methods *** + STDMETHOD(GetClassID)(CLSID *pClassID) + { + return E_NOTIMPL; + } + + // *** IStdMarshalInfo methods *** + STDMETHOD(GetClassForHandler)(DWORD dwDestContext, void *pvDestContext, CLSID *pClsid) + { + return E_NOTIMPL; + } +}; + +// Yes this sucks, but we have to support GCC. (CORE-12710) +#ifdef __REACTOS__ +#define DECLARE_QIPTR(type) CComQIIDPtr<I_ID(type)> +#else +#define DECLARE_QIPTR(type) CComQIPtr<type> +#endif + +START_TEST(CComQIPtr) +{ + CQITestObject testObject; + IUnknown* unk = static_cast<IPersist*>(&testObject); + ok(testObject.m_dwRef == 1, "Expected m_dwRef 1, got %lu\n", testObject.m_dwRef); + ok(g_QI == 0, "Expected g_QI 0, got %lu\n", g_QI); + + { + DECLARE_QIPTR(IPersist) ppPersist(unk); + ok(testObject.m_dwRef == 2, "Expected m_dwRef 2, got %lu\n", testObject.m_dwRef); + ok(g_QI == 1, "Expected g_QI 1, got %lu\n", g_QI); + + DECLARE_QIPTR(IStdMarshalInfo) ppMarshal(ppPersist); + ok(testObject.m_dwRef == 3, "Expected m_dwRef 3, got %lu\n", testObject.m_dwRef); + ok(g_QI == 2, "Expected g_QI 2, got %lu\n", g_QI); + } + ok(testObject.m_dwRef == 1, "Expected m_dwRef 1, got %lu\n", testObject.m_dwRef); + { + DECLARE_QIPTR(IStdMarshalInfo) ppMarshal; + ok(testObject.m_dwRef == 1, "Expected m_dwRef 1, got %lu\n", testObject.m_dwRef); + ok(g_QI == 2, "Expected g_QI 2, got %lu\n", g_QI); + + ppMarshal = unk; + ok(testObject.m_dwRef == 2, "Expected m_dwRef 2, got %lu\n", testObject.m_dwRef); + ok(g_QI == 3, "Expected g_QI 3, got %lu\n", g_QI); + + ppMarshal = static_cast<IUnknown*>(NULL); + ok(testObject.m_dwRef == 1, "Expected m_dwRef 1, got %lu\n", testObject.m_dwRef); + ok(g_QI == 3, "Expected g_QI 3, got %lu\n", g_QI); + + CComPtr<IUnknown> spUnk(unk); + ok(testObject.m_dwRef == 2, "Expected m_dwRef 2, got %lu\n", testObject.m_dwRef); + ok(g_QI == 3, "Expected g_QI 3, got %lu\n", g_QI); + + ppMarshal = spUnk; + ok(testObject.m_dwRef == 3, "Expected m_dwRef 3, got %lu\n", testObject.m_dwRef); + ok(g_QI == 4, "Expected g_QI 4, got %lu\n", g_QI); + + spUnk.Release(); + ok(testObject.m_dwRef == 2, "Expected m_dwRef 2, got %lu\n", testObject.m_dwRef); + ok(g_QI == 4, "Expected g_QI 4, got %lu\n", g_QI); + + spUnk = ppMarshal; + ok(testObject.m_dwRef == 3, "Expected m_dwRef 3, got %lu\n", testObject.m_dwRef); +#ifdef __REACTOS__ + // CORE-12710 + todo_if(1) +#endif + ok(g_QI == 5, "Expected g_QI 5, got %lu\n", g_QI); + } + +#ifndef __REACTOS__ + printf("CComQIPtr: %i tests executed (0 marked as todo, %i failures), 0 skipped.\n", g_tests_executed, g_tests_failed); + return g_tests_failed; +#endif +} diff --git a/modules/rostests/apitests/atl/CMakeLists.txt b/modules/rostests/apitests/atl/CMakeLists.txt index fbc1420dec..3cbf35bc09 100644 --- a/modules/rostests/apitests/atl/CMakeLists.txt +++ b/modules/rostests/apitests/atl/CMakeLists.txt @@ -9,6 +9,7 @@ list(APPEND SOURCE CComBSTR.cpp CComHeapPtr.cpp CComObject.cpp + CComQIPtr.cpp CImage.cpp CRegKey.cpp CSimpleArray.cpp diff --git a/modules/rostests/apitests/atl/devenv/ATLTest.sln b/modules/rostests/apitests/atl/devenv/ATLTest.sln index d9d18dddb6..b36db07333 100644 --- a/modules/rostests/apitests/atl/devenv/ATLTest.sln +++ b/modules/rostests/apitests/atl/devenv/ATLTest.sln @@ -13,6 +13,8 @@ Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "CString", "CString.vcxproj" EndProject Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "CComObject", "CComObject.vcxproj", "{408AEF87-D169-4A2F-A9E3-FF3DD1D59E65}" EndProject +Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "CComQIPtr", "CComQIPtr.vcxproj", "{907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}" +EndProject Global GlobalSection(SolutionConfigurationPlatforms) = preSolution Debug|x64 = Debug|x64 @@ -61,6 +63,14 @@ Global {408AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x64.Build.0 = Release|x64 {408AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x86.ActiveCfg = Release|Win32 {408AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x86.Build.0 = Release|Win32 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Debug|x64.ActiveCfg = Debug|x64 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Debug|x64.Build.0 = Debug|x64 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Debug|x86.ActiveCfg = Debug|Win32 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Debug|x86.Build.0 = Debug|Win32 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x64.ActiveCfg = Release|x64 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x64.Build.0 = Release|x64 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x86.ActiveCfg = Release|Win32 + {907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}.Release|x86.Build.0 = Release|Win32 EndGlobalSection GlobalSection(SolutionProperties) = preSolution HideSolutionNode = FALSE diff --git a/modules/rostests/apitests/atl/devenv/CComObject.vcxproj b/modules/rostests/apitests/atl/devenv/CComObject.vcxproj index ac525e52d9..97903a9250 100644 --- a/modules/rostests/apitests/atl/devenv/CComObject.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CComObject.vcxproj @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,7 +156,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> diff --git a/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj b/modules/rostests/apitests/atl/devenv/CComQIPtr.vcxproj similarity index 96% copy from modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj copy to modules/rostests/apitests/atl/devenv/CComQIPtr.vcxproj index f7e7549036..345f1d8ea1 100644 --- a/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CComQIPtr.vcxproj @@ -19,7 +19,7 @@ </ProjectConfiguration> </ItemGroup> <PropertyGroup Label="Globals"> - <ProjectGuid>{EC560DE6-6DB3-437D-85CA-582491FE6F95}</ProjectGuid> + <ProjectGuid>{907AEF87-D169-4A2F-A9E3-FF3DD1D59E65}</ProjectGuid> <WindowsTargetPlatformVersion>8.1</WindowsTargetPlatformVersion> <Keyword>AtlProj</Keyword> </PropertyGroup> @@ -39,8 +39,8 @@ <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'" Label="Configuration"> <ConfigurationType>Application</ConfigurationType> <UseDebugLibraries>true</UseDebugLibraries> - <PlatformToolset>v120_xp</PlatformToolset> <CharacterSet>Unicode</CharacterSet> + <PlatformToolset>v120_xp</PlatformToolset> </PropertyGroup> <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'" Label="Configuration"> <ConfigurationType>Application</ConfigurationType> @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,11 +156,10 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> - <ClCompile Include="../CSimpleMap.cpp"> + <ClCompile Include="../CComQIPtr.cpp"> <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">MultiThreaded</RuntimeLibrary> <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Release|x64'">MultiThreaded</RuntimeLibrary> <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">MultiThreadedDebug</RuntimeLibrary> diff --git a/modules/rostests/apitests/atl/devenv/CImage.vcxproj b/modules/rostests/apitests/atl/devenv/CImage.vcxproj index 5c77112862..80984b87c9 100644 --- a/modules/rostests/apitests/atl/devenv/CImage.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CImage.vcxproj @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,7 +156,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> diff --git a/modules/rostests/apitests/atl/devenv/CSimpleArray.vcxproj b/modules/rostests/apitests/atl/devenv/CSimpleArray.vcxproj index 8053eea219..76b03d7598 100644 --- a/modules/rostests/apitests/atl/devenv/CSimpleArray.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CSimpleArray.vcxproj @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,7 +156,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> diff --git a/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj b/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj index f7e7549036..03862264d7 100644 --- a/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CSimpleMap.vcxproj @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,7 +156,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> diff --git a/modules/rostests/apitests/atl/devenv/CString.vcxproj b/modules/rostests/apitests/atl/devenv/CString.vcxproj index 82dfc4687b..8772a16814 100644 --- a/modules/rostests/apitests/atl/devenv/CString.vcxproj +++ b/modules/rostests/apitests/atl/devenv/CString.vcxproj @@ -98,7 +98,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>false</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> @@ -117,7 +116,6 @@ <Link> <SubSystem>Console</SubSystem> <GenerateDebugInformation>true</GenerateDebugInformation> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> @@ -138,7 +136,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> @@ -159,7 +156,6 @@ <GenerateDebugInformation>true</GenerateDebugInformation> <EnableCOMDATFolding>true</EnableCOMDATFolding> <OptimizeReferences>true</OptimizeReferences> - <RegisterOutput>true</RegisterOutput> </Link> </ItemDefinitionGroup> <ItemGroup> diff --git a/modules/rostests/apitests/atl/testlist.c b/modules/rostests/apitests/atl/testlist.c index 0c40c3b140..232785a4b8 100644 --- a/modules/rostests/apitests/atl/testlist.c +++ b/modules/rostests/apitests/atl/testlist.c @@ -5,6 +5,7 @@ extern void func_atltypes(void); extern void func_CComBSTR(void); extern void func_CComHeapPtr(void); extern void func_CComObject(void); +extern void func_CComQIPtr(void); extern void func_CComVariant(void); extern void func_CImage(void); extern void func_CRegKey(void); @@ -18,6 +19,7 @@ const struct test winetest_testlist[] = { "CComBSTR", func_CComBSTR }, { "CComHeapPtr", func_CComHeapPtr }, { "CComObject", func_CComObject }, + { "CComQIPtr", func_CComQIPtr }, { "CComVariant", func_CComVariant }, { "CImage", func_CImage }, { "CRegKey", func_CRegKey },

6 years, 4 months

01/02: [ATL] Improve the order of operations in CComPtr and CComQIIDPtr

by Mark Jansen

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=69893ccf3824da558e75d… commit 69893ccf3824da558e75d376d3f6ac2fb1292711 Author: Mark Jansen <mark.jansen(a)reactos.org> AuthorDate: Sun Aug 5 11:48:31 2018 +0200 Commit: Mark Jansen <mark.jansen(a)reactos.org> CommitDate: Sun Aug 19 17:35:18 2018 +0200 [ATL] Improve the order of operations in CComPtr and CComQIIDPtr --- sdk/lib/atl/atlcomcli.h | 60 +++++++++++++++++++++++++++++++++++++++---------- 1 file changed, 48 insertions(+), 12 deletions(-) diff --git a/sdk/lib/atl/atlcomcli.h b/sdk/lib/atl/atlcomcli.h index 229c585fdd..ad1582a6ca 100644 --- a/sdk/lib/atl/atlcomcli.h +++ b/sdk/lib/atl/atlcomcli.h @@ -87,23 +87,49 @@ public: T *operator = (T *lp) { - if (p != NULL) - p->Release(); + T* pOld = p; + p = lp; if (p != NULL) p->AddRef(); + + if (pOld != NULL) + pOld->Release(); + return *this; } T *operator = (const CComPtr<T> &lp) { - if (p != NULL) - p->Release(); + T* pOld = p; + p = lp.p; if (p != NULL) p->AddRef(); + + if (pOld != NULL) + pOld->Release(); + + return *this; + } + + // We cannot enable this until gcc starts supporting __uuidof + // See CORE-12710 +#if 0 + template <typename Q> + T* operator=(const CComPtr<Q>& lp) + { + T* pOld = p; + + if (!lp.p || FAILED(lp.p->QueryInterface(__uuidof(T), (void**)(IUnknown**)&p))) + p = NULL; + + if (pOld != NULL) + pOld->Release(); + return *this; } +#endif void Release() { @@ -175,38 +201,48 @@ public: { if (lp != NULL) { - if (FAILED(lp->QueryInterface(*piid, reinterpret_cast<void **>(&p)))) + if (FAILED(lp->QueryInterface(*piid, (void**)(IUnknown**)&p))) p = NULL; } } T *operator = (T *lp) { - if (p != NULL) - p->Release(); + T* pOld = p; + p = lp; if (p != NULL) p->AddRef(); + + if (pOld != NULL) + pOld->Release(); + return *this; } T *operator = (const CComQIIDPtr<T,piid> &lp) { - if (p != NULL) - p->Release(); + T* pOld = p; + p = lp.p; if (p != NULL) p->AddRef(); + + if (pOld != NULL) + pOld->Release(); + return *this; } T * operator=(IUnknown* lp) { - if (p != NULL) - p->Release(); + T* pOld = p; - if (FAILED(lp->QueryInterface(*piid, reinterpret_cast<void **>(&p)))) + if (!lp || FAILED(lp->QueryInterface(*piid, (void**)(IUnknown**)&p))) p = NULL; + if (pOld != NULL) + pOld->Release(); + return *this; } };

6 years, 4 months

01/01: [WSHOM.OCX] Zero-initialize timeout and type if they are not set. This should fix buildbot timeouts

by EUPHORIA-IT\m.jansen

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=bb5f8cfccb4a716edc2ab… commit bb5f8cfccb4a716edc2ab6e723b95693ec53f2a1 Author: EUPHORIA-IT\m.jansen <m.jansen(a)euphoria-it.nl> AuthorDate: Sun Aug 19 16:13:55 2018 +0200 Commit: EUPHORIA-IT\m.jansen <m.jansen(a)euphoria-it.nl> CommitDate: Sun Aug 19 16:14:08 2018 +0200 [WSHOM.OCX] Zero-initialize timeout and type if they are not set. This should fix buildbot timeouts --- dll/win32/wshom.ocx/shell.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/dll/win32/wshom.ocx/shell.c b/dll/win32/wshom.ocx/shell.c index 1aa353a17a..42310e2a7e 100644 --- a/dll/win32/wshom.ocx/shell.c +++ b/dll/win32/wshom.ocx/shell.c @@ -1316,6 +1316,12 @@ static HRESULT WINAPI WshShell3_Popup(IWshShell3 *iface, BSTR text, VARIANT *sec if (FAILED(hr)) return hr; } +#ifdef __REACTOS__ + else + { + VariantChangeType(&timeout, &timeout, 0, VT_I4); + } +#endif VariantInit(&param.type); if (!is_optional_argument(type)) @@ -1324,6 +1330,12 @@ static HRESULT WINAPI WshShell3_Popup(IWshShell3 *iface, BSTR text, VARIANT *sec if (FAILED(hr)) return hr; } +#ifdef __REACTOS__ + else + { + VariantChangeType(&param.type, &param.type, 0, VT_I4); + } +#endif if (is_optional_argument(title)) param.title = *title;

6 years, 4 months

01/01: [SYSSETUP] SetupStartService: Remove the wait loop

by Eric Kohl

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a8cf35d0e4333d1ca0b69… commit a8cf35d0e4333d1ca0b694602629c7dc7408944d Author: Eric Kohl <eric.kohl(a)reactos.org> AuthorDate: Sun Aug 19 14:31:43 2018 +0200 Commit: Eric Kohl <eric.kohl(a)reactos.org> CommitDate: Sun Aug 19 14:32:29 2018 +0200 [SYSSETUP] SetupStartService: Remove the wait loop CORE-14909 --- dll/win32/syssetup/install.c | 26 ++------------------------ 1 file changed, 2 insertions(+), 24 deletions(-) diff --git a/dll/win32/syssetup/install.c b/dll/win32/syssetup/install.c index e9b5745def..63fa4a28e9 100644 --- a/dll/win32/syssetup/install.c +++ b/dll/win32/syssetup/install.c @@ -1508,9 +1508,7 @@ SetupStartService( { SC_HANDLE hManager = NULL; SC_HANDLE hService = NULL; - SERVICE_STATUS ServiceStatus; DWORD dwError = ERROR_SUCCESS; - DWORD dwRetries = 0; hManager = OpenSCManagerW(NULL, NULL, @@ -1523,7 +1521,7 @@ SetupStartService( hService = OpenServiceW(hManager, lpServiceName, - SERVICE_START | (bWait) ? SERVICE_QUERY_STATUS : 0); + SERVICE_START); if (hService == NULL) { dwError = GetLastError(); @@ -1533,27 +1531,7 @@ SetupStartService( if (!StartService(hService, 0, NULL)) { dwError = GetLastError(); - if (dwError != ERROR_SERVICE_ALREADY_RUNNING) - goto done; - - if (bWait) - { - for (;;) - { - if (!QueryServiceStatus(hService, &ServiceStatus)) - break; - - if (ServiceStatus.dwCurrentState != SERVICE_START_PENDING) - break; - - if (dwRetries == 30) - break; - - dwRetries++; - - Sleep(5000); - } - } + goto done; } done:

6 years, 4 months

01/01: [NTDLL][NTUSER] Fix unselected text after WM_CBLOSTTEXTFOCUS

by Manuel Bachmann

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=71ab0b5d4cb7fe00e0047… commit 71ab0b5d4cb7fe00e0047308a18ae48c734266b9 Author: Manuel Bachmann <tarnyko(a)tarnyko.net> AuthorDate: Thu Nov 2 02:06:50 2017 +0100 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Sun Aug 19 14:24:34 2018 +0200 [NTDLL][NTUSER] Fix unselected text after WM_CBLOSTTEXTFOCUS When we unselect text after the WM_CBLOSTTEXTFOCUS message, make sure we also forget we have been focused at all; otherwise the edit may become focused again, but with an empty text selection. CORE-10266 --- win32ss/user/user32/controls/combo.c | 16 +++++++--------- win32ss/user/user32/controls/edit.c | 11 +---------- 2 files changed, 8 insertions(+), 19 deletions(-) diff --git a/win32ss/user/user32/controls/combo.c b/win32ss/user/user32/controls/combo.c index eb9d6fd86d..11537901e5 100644 --- a/win32ss/user/user32/controls/combo.c +++ b/win32ss/user/user32/controls/combo.c @@ -2260,16 +2260,14 @@ LRESULT WINAPI ComboWndProc_common( HWND hwnd, UINT message, WPARAM wParam, LPAR } break; - case WM_CBLOSTTEXTFOCUS: /* undocumented message - deselects the text when focus is lost */ + case WM_CBLOSTTEXTFOCUS: /* undocumented message - deselects the text when focus is lost */ + if (lphc->hWndEdit != NULL) { - if (lphc->hWndEdit != NULL) - { - SendMessage(lphc->self, WM_LBUTTONUP, 0, 0xFFFFFFFF); - SendMessage(lphc->hWndEdit, EM_SETSEL, 0, 0); - lphc->wState &= ~CBF_FOCUSED; - CB_NOTIFY(lphc, CBN_KILLFOCUS); - } - } + SendMessage(lphc->self, WM_LBUTTONUP, 0, 0xFFFFFFFF); + SendMessage(lphc->hWndEdit, EM_SETSEL, 0, 0); + lphc->wState &= ~(CBF_FOCUSED | CBF_BEENFOCUSED); + CB_NOTIFY(lphc, CBN_KILLFOCUS); + } return TRUE; #endif diff --git a/win32ss/user/user32/controls/edit.c b/win32ss/user/user32/controls/edit.c index f1db16f8e0..70bdd1d40d 100644 --- a/win32ss/user/user32/controls/edit.c +++ b/win32ss/user/user32/controls/edit.c @@ -3626,7 +3626,6 @@ static LRESULT EDIT_WM_KeyDown(EDITSTATE *es, INT key) */ static LRESULT EDIT_WM_KillFocus(EDITSTATE *es) { -#if 0 // See CORE-10266. HWND hCombo; LONG lStyles; @@ -3647,15 +3646,7 @@ static LRESULT EDIT_WM_KillFocus(EDITSTATE *es) if ((lStyles & CBS_DROPDOWN) || (lStyles & CBS_SIMPLE)) SendMessage(hCombo, WM_CBLOSTTEXTFOCUS, 0, 0); } -#else - es->flags &= ~EF_FOCUSED; - DestroyCaret(); - if(!(es->style & ES_NOHIDESEL)) - EDIT_InvalidateText(es, es->selection_start, es->selection_end); - EDIT_NOTIFY_PARENT(es, EN_KILLFOCUS); - /* throw away left over scroll when we lose focus */ - es->wheelDeltaRemainder = 0; -#endif + return 0; }

6 years, 4 months

02/02: [WINED3D] Yield CPU to the scheduler in wined3d_pause. CORE-14637 CORE-14534

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c97e9defc157b171d56e3… commit c97e9defc157b171d56e31dd9768c5eee732019d Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sun Aug 19 13:28:06 2018 +0200 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Sun Aug 19 13:46:08 2018 +0200 [WINED3D] Yield CPU to the scheduler in wined3d_pause. CORE-14637 CORE-14534 Much like the previous commit, this is a workaround for our buggy Win32k. --- dll/directx/wine/wined3d/wined3d_private.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dll/directx/wine/wined3d/wined3d_private.h b/dll/directx/wine/wined3d/wined3d_private.h index 3444b52928..3cf6748237 100644 --- a/dll/directx/wine/wined3d/wined3d_private.h +++ b/dll/directx/wine/wined3d/wined3d_private.h @@ -370,9 +370,13 @@ static inline unsigned int wined3d_popcount(unsigned int x) static inline void wined3d_pause(void) { +#ifdef __REACTOS__ + Sleep(0); +#else #if defined(__GNUC__) && (defined(__i386__) || defined(__x86_64__)) __asm__ __volatile__( "rep;nop" : : : "memory" ); #endif +#endif } #define ORM_BACKBUFFER 0

6 years, 4 months

01/02: [WINED3D] Avoid spinning in wined3d_cs_run. CORE-14534

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=adfbefde2b237f757d741… commit adfbefde2b237f757d741e0cf17197e51599d436 Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sun Aug 19 13:27:04 2018 +0200 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Sun Aug 19 13:46:02 2018 +0200 [WINED3D] Avoid spinning in wined3d_cs_run. CORE-14534 Spinning here is supposed to improve performance by avoiding scheduling overhead, but because ReactOS's Win32 subsystem does not deal with this well, it causes the system to freeze on ROS. As a workaround, we disable spinning and jump directly to the wait. It should become obvious when this starts actually causing performance issues again and it's time to undo. --- dll/directx/wine/wined3d/wined3d_private.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dll/directx/wine/wined3d/wined3d_private.h b/dll/directx/wine/wined3d/wined3d_private.h index 7b3731c014..3444b52928 100644 --- a/dll/directx/wine/wined3d/wined3d_private.h +++ b/dll/directx/wine/wined3d/wined3d_private.h @@ -3544,7 +3544,11 @@ enum wined3d_push_constants #define WINED3D_CS_QUERY_POLL_INTERVAL 10u #define WINED3D_CS_QUEUE_SIZE 0x100000u +#ifdef __REACTOS__ +#define WINED3D_CS_SPIN_COUNT 1u +#else #define WINED3D_CS_SPIN_COUNT 10000000u +#endif struct wined3d_cs_queue {

6 years, 4 months

01/01: [USER32] Log the text of message boxes to help figure out some test bot timeouts.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=9bffdcb6e7e6f6e132488… commit 9bffdcb6e7e6f6e1324886daa0ad6d3afc3b03c2 Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sun Aug 19 12:02:17 2018 +0200 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Sun Aug 19 12:02:24 2018 +0200 [USER32] Log the text of message boxes to help figure out some test bot timeouts. --- win32ss/user/user32/windows/messagebox.c | 1 + 1 file changed, 1 insertion(+) diff --git a/win32ss/user/user32/windows/messagebox.c b/win32ss/user/user32/windows/messagebox.c index 658ba763b4..57033ad4e6 100644 --- a/win32ss/user/user32/windows/messagebox.c +++ b/win32ss/user/user32/windows/messagebox.c @@ -1045,6 +1045,7 @@ SoftModalMessageBox(IN LPMSGBOXDATA lpMsgBoxData) tpl->cy = RESCALE_Y(btntop, units); /* Finally show the message-box */ + ERR("MessageBox: %s\n", wine_dbgstr_wn(lpMsgBoxParams->lpszText, textlen)); ret = DialogBoxIndirectParamW(lpMsgBoxParams->hInstance, tpl, lpMsgBoxParams->hwndOwner, MessageBoxProc, (LPARAM)&mbd);

6 years, 4 months

← Newer
1
...
11
12
13
14
15
16
17
...
31
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs August 2018