Ros-diffs April 2015

ros-diffs@reactos.org

17 participants
481 discussions

[hbelusca] 67054: [CMD] - Remove useless STRING_START_ERROR1. - Fix french translation.

by hbelusca＠svn.reactos.org

Author: hbelusca Date: Sun Apr 5 00:26:12 2015 New Revision: 67054 URL: http://svn.reactos.org/svn/reactos?rev=67054&view=rev Log: [CMD] - Remove useless STRING_START_ERROR1. - Fix french translation. Modified: trunk/reactos/base/shell/cmd/lang/de-DE.rc trunk/reactos/base/shell/cmd/lang/el-GR.rc trunk/reactos/base/shell/cmd/lang/en-US.rc trunk/reactos/base/shell/cmd/lang/es-ES.rc trunk/reactos/base/shell/cmd/lang/fr-FR.rc trunk/reactos/base/shell/cmd/lang/hu-HU.rc trunk/reactos/base/shell/cmd/lang/id-ID.rc trunk/reactos/base/shell/cmd/lang/it-IT.rc trunk/reactos/base/shell/cmd/lang/ja-JP.rc trunk/reactos/base/shell/cmd/lang/no-NO.rc trunk/reactos/base/shell/cmd/lang/pl-PL.rc trunk/reactos/base/shell/cmd/lang/ro-RO.rc trunk/reactos/base/shell/cmd/lang/ru-RU.rc trunk/reactos/base/shell/cmd/lang/sk-SK.rc trunk/reactos/base/shell/cmd/lang/sq-AL.rc trunk/reactos/base/shell/cmd/lang/sv-SE.rc trunk/reactos/base/shell/cmd/lang/uk-UA.rc trunk/reactos/base/shell/cmd/lang/zh-CN.rc trunk/reactos/base/shell/cmd/resource.h Modified: trunk/reactos/base/shell/cmd/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/de-DE.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/de-DE.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/de-DE.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -495,7 +495,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[FEHLER]\n" STRING_REN_ERROR1 "MoveFile() ist fehlgeschlagen. Fehler: %lu\n" - STRING_START_ERROR1 "Derzeit noch kein Batch Support!" STRING_TIME_ERROR1 "UngÃ¼ltige Zeit." STRING_TYPE_ERROR1 "UngÃ¼ltige Option '/%s'\n" STRING_WINDOW_ERROR1 "Fenster nicht gefunden" Modified: trunk/reactos/base/shell/cmd/lang/el-GR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/el-GR.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/el-GR.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/el-GR.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -500,7 +500,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Î£ÏÎ¬Î»Î¼Î±]\n" STRING_REN_ERROR1 "MoveFile() failed. Error: %lu\n" - STRING_START_ERROR1 "No batch support at the moment!" STRING_TIME_ERROR1 "ÎÎ· ÎÎ³ÎºÏÏÎ· ÏÏÎ±." STRING_TYPE_ERROR1 "ÎÎ· ÎÎ³ÎºÏÏÎ· ÎµÏÎ¹Î»Î¿Î³Î® '/%s'\n" STRING_WINDOW_ERROR1 "ÏÎ¿ ÏÎ±ÏÎ¬Î¸ÏÏÎ¿ Î´Îµ Î²ÏÎÎ¸Î·ÎºÎµ" Modified: trunk/reactos/base/shell/cmd/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/en-US.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/en-US.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/en-US.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -496,7 +496,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Error]\n" STRING_REN_ERROR1 "MoveFile() failed. Error: %lu\n" - STRING_START_ERROR1 "No batch support at the moment!" STRING_TIME_ERROR1 "Invalid time." STRING_TYPE_ERROR1 "Invalid option '/%s'\n" STRING_WINDOW_ERROR1 "window not found" Modified: trunk/reactos/base/shell/cmd/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/es-ES.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/es-ES.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/es-ES.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -498,7 +498,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Error]\n" STRING_REN_ERROR1 "MoveFile() fallÃ³. Error: %lu\n" - STRING_START_ERROR1 "Â¡No hay soporte para archivos por lotes en este momento!" STRING_TIME_ERROR1 "Hora incorrecta." STRING_TYPE_ERROR1 "OpciÃ³n incorrecta '/%s'\n" STRING_WINDOW_ERROR1 "Ventana no encontrada" Modified: trunk/reactos/base/shell/cmd/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/fr-FR.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/fr-FR.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/fr-FR.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -510,7 +510,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Erreur]\n" STRING_REN_ERROR1 "MoveFile() a Ã©chouÃ©. Erreur : %lu\n" - STRING_START_ERROR1 "Pas de support batch pour l'instant !" STRING_TIME_ERROR1 "Heure invalide." STRING_TYPE_ERROR1 "Option invalide '/%s'\n" STRING_WINDOW_ERROR1 "FenÃªtre non trouvÃ©e" @@ -524,7 +523,7 @@ STRING_ERROR_INVALID_PARAM_FORMAT "Format de paramÃ¨tre incorrect - %s\n" STRING_ERROR_BADCOMMAND "Commande ou nom de fichier inconnu - %s\n" STRING_ERROR_OUT_OF_MEMORY "Pas assez de mÃ©moire.\n" - STRING_ERROR_CANNOTPIPE "Erreur ! Impossible de chaÃ®ner ! Ãchec Ã l'ouverture du fichier temporaire!\n" + STRING_ERROR_CANNOTPIPE "Erreur ! Impossible de chaÃ®ner ! Ãchec Ã l'ouverture du fichier temporaire!\n" STRING_ERROR_D_PAUSEMSG "Appuyez sur une touche pour continuer . . ." STRING_ERROR_DRIVER_NOT_READY "Lecteur non prÃªt" STRING_PATH_ERROR "CMD : Pas dans l'environnement '%s'\n" @@ -546,11 +545,11 @@ STRING_SCREEN_COL "valeur invalide pour col" STRING_SCREEN_ROW "valeur invalide pour lig" STRING_TIMER_TIME "Le timer nÂ°%d est %s : " - STRING_MKLINK_CREATED_SYMBOLIC "Symbolic link created for %s <<===>> %s\n" - STRING_MKLINK_CREATED_HARD "Hard link created for %s <<===>> %s\n" - STRING_MKLINK_CREATED_JUNCTION "Junction created for %s <<===>> %s\n" - STRING_MORE "More? " - STRING_CANCEL_BATCH_FILE "\r\nCtrl-Break pressed. Cancel batch file? (Oui/Non/Tous) " + STRING_MKLINK_CREATED_SYMBOLIC "Lien symbolique crÃ©Ã© pour %s <<===>> %s\n" + STRING_MKLINK_CREATED_HARD "Lien rÃ©el crÃ©Ã© pour %s <<===>> %s\n" + STRING_MKLINK_CREATED_JUNCTION "Jonction crÃ©Ã©e pour %s <<===>> %s\n" + STRING_MORE "Plus ? " + STRING_CANCEL_BATCH_FILE "\r\nAppui sur Ctrl-Break. Terminer le programme batch ? (Oui/Non/Tous) " STRING_INVALID_OPERAND "OpÃ©rande invalide." STRING_EXPECTED_CLOSE_PAREN "Attendu ')'." STRING_EXPECTED_NUMBER_OR_VARIABLE "Attendu nombre ou nom de variable." Modified: trunk/reactos/base/shell/cmd/lang/hu-HU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/hu-HU.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/hu-HU.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/hu-HU.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -480,7 +480,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[HIBA]\n" STRING_REN_ERROR1 "MoveÃllomÃ¡ny() sikertelen. Hiba: %lu\n" - STRING_START_ERROR1 "Nincs batch tÃ¡mogatÃ¡s jelenleg!" STRING_TIME_ERROR1 "ÃrvÃ©nytelen idÃµ." STRING_TYPE_ERROR1 "ÃrvÃ©nytelen beÃ¡llÃtÃ¡s '/%s'\n" STRING_WINDOW_ERROR1 "Az ablak nem talÃ¡lhatÃ³" Modified: trunk/reactos/base/shell/cmd/lang/id-ID.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/id-ID.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/id-ID.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/id-ID.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -495,7 +495,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Salah]\n" STRING_REN_ERROR1 "MoveFile() gagal. Kesalahan: %lu\n" - STRING_START_ERROR1 "Tidak ada dukungan batch untuk saat ini!" STRING_TIME_ERROR1 "Jam tidak benar." STRING_TYPE_ERROR1 "Opsi tidak benar '/%s'\n" STRING_WINDOW_ERROR1 "jendela tidak ditemukan" Modified: trunk/reactos/base/shell/cmd/lang/it-IT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/it-IT.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/it-IT.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/it-IT.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -506,7 +506,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Errore]\n" STRING_REN_ERROR1 "MoveFile() fallita. Errore: %lu\n" - STRING_START_ERROR1 "Nessun supporto ai batch al momento!" STRING_TIME_ERROR1 "Ora non valida." STRING_TYPE_ERROR1 "Opzione '/%s' non valida\n" STRING_WINDOW_ERROR1 "finestra non trovata" Modified: trunk/reactos/base/shell/cmd/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/ja-JP.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/ja-JP.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/ja-JP.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -503,7 +503,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[ã¨ã©ã¼]\n" STRING_REN_ERROR1 "MoveFile() failed. ã¨ã©ã¼: %lu\n" - STRING_START_ERROR1 "ç¾æç¹ã§ã¯ãããã¯ãµãã¼ãããã¦ãã¾ãã!" STRING_TIME_ERROR1 "ç¡å¹ãªæå»ã§ãã" STRING_TYPE_ERROR1 "'/%s' ã¯ç¡å¹ãªãªãã·ã§ã³ã§ã\n" STRING_WINDOW_ERROR1 "ã¦ã£ã³ãã¦ãè¦ã¤ããã¾ããã" Modified: trunk/reactos/base/shell/cmd/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/no-NO.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/no-NO.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/no-NO.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -494,7 +494,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Feil]\n" STRING_REN_ERROR1 "MoveFile() mislykkes. Feil: %lu\n" - STRING_START_ERROR1 "Ingen satsvis stÃ¸tte for Ã¸yeblikket!" STRING_TIME_ERROR1 "Ugyldig tid." STRING_TYPE_ERROR1 "Ugyldig alternativ '/%s'\n" STRING_WINDOW_ERROR1 "vinduet ble ikke funnet" Modified: trunk/reactos/base/shell/cmd/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/pl-PL.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/pl-PL.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/pl-PL.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -500,7 +500,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[BÅÄd]\n" STRING_REN_ERROR1 "MoveFile() nieudane. BÅÄd: %lu\n" - STRING_START_ERROR1 "Programy wsadowe nie zostaÅy jeszcze zaimplementowane!" STRING_TIME_ERROR1 "NieprawidÅowy czas." STRING_TYPE_ERROR1 "NieprawidÅowa opcja '/%s'\n" STRING_WINDOW_ERROR1 "Okno nie zostaÅo odnalezione" Modified: trunk/reactos/base/shell/cmd/lang/ro-RO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/ro-RO.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/ro-RO.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/ro-RO.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -531,7 +531,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Eroare]\n" STRING_REN_ERROR1 "A eÈuat MoveFile(). Eroarea: %lu\n" - STRING_START_ERROR1 "DeocamdatÄ fiÈierele de comenzi nu sunt admise!" STRING_TIME_ERROR1 "OrÄ nevalidÄ." STRING_TYPE_ERROR1 "OpÈiune eronatÄ Â«/%sÂ»\n" STRING_WINDOW_ERROR1 "fereastra nu a fost gÄsitÄ" Modified: trunk/reactos/base/shell/cmd/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/ru-RU.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/ru-RU.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/ru-RU.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -505,7 +505,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[ÐÑÐ¸Ð±ÐºÐ°]\n" STRING_REN_ERROR1 "ÐÑÐ¿Ð¾Ð»Ð½ÐµÐ½Ð¸Ðµ MoveFile() Ð±ÑÐ»Ð¾ Ð½Ðµ ÑÐ´Ð°ÑÐ½ÑÐ¼. ÐÑÐ¸Ð±ÐºÐ°: %lu\n" - STRING_START_ERROR1 "ÐÐ° Ð´Ð°Ð½Ð½ÑÐ¹ Ð¼Ð¾Ð¼ÐµÐ½Ñ Ð½ÐµÑ Ð¿Ð¾Ð´Ð´ÐµÑÐ¶ÐºÐ¸ Ð¿Ð°ÐºÐµÑÐ½ÑÑ ÑÐ°Ð¹Ð»Ð¾Ð²!" STRING_TIME_ERROR1 "ÐÐµÐ²ÐµÑÐ½Ð¾Ðµ Ð²ÑÐµÐ¼Ñ." STRING_TYPE_ERROR1 "ÐÐµÐ²ÐµÑÐ½ÑÐ¹ Ð¿Ð°ÑÐ°Ð¼ÐµÑÑ '/%s'\n" STRING_WINDOW_ERROR1 "Ð¾ÐºÐ½Ð¾ Ð½Ðµ Ð½Ð°Ð¹Ð´ÐµÐ½Ð¾" Modified: trunk/reactos/base/shell/cmd/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/sk-SK.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/sk-SK.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/sk-SK.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -502,7 +502,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Chyba]\n" STRING_REN_ERROR1 "MoveFile() failed. Error: %lu\n" - STRING_START_ERROR1 "No batch support at the moment!" STRING_TIME_ERROR1 "NeplatnÃ½ Äas." STRING_TYPE_ERROR1 "Invalid option '/%s'\n" STRING_WINDOW_ERROR1 "okno sa nenaÅ¡lo" Modified: trunk/reactos/base/shell/cmd/lang/sq-AL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/sq-AL.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/sq-AL.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/sq-AL.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -500,7 +500,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Error]\n" STRING_REN_ERROR1 "MoveFile() dÃ«shtoj. Error: %lu\n" - STRING_START_ERROR1 "Nuk ka mbÃ«shtetje nÃ« kÃ«tÃ« moment bÃ«r dokumentin batch!" STRING_TIME_ERROR1 "Ora invalide." STRING_TYPE_ERROR1 "opsion ivalid '/%s'\n" STRING_WINDOW_ERROR1 "dritaria nuk i gjend" Modified: trunk/reactos/base/shell/cmd/lang/sv-SE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/sv-SE.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/sv-SE.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/sv-SE.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -494,7 +494,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[Feil]\n" STRING_REN_ERROR1 "MoveFile() mislykkes. Feil: %lu\n" - STRING_START_ERROR1 "Ingen satsvis stÃ¸tte for Ã¸yeblikket!" STRING_TIME_ERROR1 "Ugyldig tid." STRING_TYPE_ERROR1 "Ugyldig alternativ '/%s'\n" STRING_WINDOW_ERROR1 "vinduet ble inte funnet" Modified: trunk/reactos/base/shell/cmd/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/uk-UA.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/uk-UA.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/uk-UA.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -506,7 +506,6 @@ STRING_MOVE_ERROR1 "[OK]\n" STRING_MOVE_ERROR2 "[ÐÐ¾Ð¼Ð¸Ð»ÐºÐ°]\n" STRING_REN_ERROR1 "ÐÐµÐ²Ð´Ð°ÑÐ½Ðµ Ð²Ð¸ÐºÐ¾Ð½Ð°Ð½Ð½Ñ MoveFile(). ÐÐ¾Ð¼Ð¸Ð»ÐºÐ°: %lu\n" - STRING_START_ERROR1 "ÐÐ° Ð´Ð°Ð½Ð¸Ð¹ Ð¼Ð¾Ð¼ÐµÐ½Ñ Ð¿Ð°ÐºÐµÑÐ½i ÑÐ°Ð¹Ð»Ð¸ Ð½Ðµ Ð¿iÐ´ÑÑÐ¸Ð¼ÑÑÑÑÑÑ!" STRING_TIME_ERROR1 "ÐÐµÐ²iÑÐ½Ð¸Ð¹ ÑÐ°Ñ." STRING_TYPE_ERROR1 "ÐÐµÐ²iÑÐ½Ð¸Ð¹ Ð¿Ð°ÑÐ°Ð¼ÐµÑÑ '/%s'\n" STRING_WINDOW_ERROR1 "Ð²iÐºÐ½Ð¾ Ð½Ðµ Ð·Ð½Ð°Ð¹Ð´ÐµÐ½Ð¾" Modified: trunk/reactos/base/shell/cmd/lang/zh-CN.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/lang/zh-CN.… ============================================================================== --- trunk/reactos/base/shell/cmd/lang/zh-CN.rc [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/lang/zh-CN.rc [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -465,7 +465,6 @@ STRING_MOVE_ERROR1 "[æå]\n" STRING_MOVE_ERROR2 "[éè¯¯]\n" STRING_REN_ERROR1 "MoveFile() å¤±è´¥ãéè¯¯ç ï¼%lu\n" - STRING_START_ERROR1 "ç°å¨æ²¡ææ¹å¤çæ¯æï¼" STRING_TIME_ERROR1 "æ ææ¶é´ã" STRING_TYPE_ERROR1 "æ æéé¡¹ '/%s'\n" STRING_WINDOW_ERROR1 "æªæ¾å°çªå£" Modified: trunk/reactos/base/shell/cmd/resource.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/resource.h?… ============================================================================== --- trunk/reactos/base/shell/cmd/resource.h [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/resource.h [iso-8859-1] Sun Apr 5 00:26:12 2015 @@ -57,7 +57,6 @@ #define STRING_MOVE_ERROR2 343 #define STRING_PATH_ERROR 345 #define STRING_REN_ERROR1 346 -#define STRING_START_ERROR1 347 #define STRING_TIME_ERROR1 348 #define STRING_TYPE_ERROR1 349 #define STRING_WINDOW_ERROR1 350

9 years, 8 months

[spetreolle] 67053: [FREELDR] In a quest to better registry, don't break VSSolution builds. freeldr_pe is not in the same directory and copy doesn't care if you ask to concatenate C:\tomatoes, it a...

by spetreolle＠svn.reactos.org

Author: spetreolle Date: Sat Apr 4 20:33:18 2015 New Revision: 67053 URL: http://svn.reactos.org/svn/reactos?rev=67053&view=rev Log: [FREELDR] In a quest to better registry, don't break VSSolution builds. freeldr_pe is not in the same directory and copy doesn't care if you ask to concatenate C:\tomatoes, it already has the first file. Modified: trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt Modified: trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/CMake… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] Sat Apr 4 20:33:18 2015 @@ -226,10 +226,13 @@ add_dependencies(freeldr_pe asm) add_dependencies(freeldr_pe_dbg asm) +# Retrieve the full path to the generated file of the 'freeldr_pe' target +get_target_property(_freeldr_pe_output_file freeldr_pe LOCATION) + concatenate_files( ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin - ${CMAKE_CURRENT_BINARY_DIR}/freeldr_pe.dll) + ${_freeldr_pe_output_file}) add_custom_target(freeldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys) @@ -240,7 +243,8 @@ concatenate_files( ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin - ${CMAKE_CURRENT_BINARY_DIR}/freeldr_pe.dll) + ${_freeldr_pe_output_file}) add_custom_target(setupldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys) add_cd_file(TARGET setupldr FILE ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys DESTINATION loader NO_CAB FOR bootcd regtest) +

9 years, 8 months

[cwittich] 67052: [SHELL32] implement invert selection

by cwittich＠svn.reactos.org

Author: cwittich Date: Sat Apr 4 18:51:28 2015 New Revision: 67052 URL: http://svn.reactos.org/svn/reactos?rev=67052&view=rev Log: [SHELL32] implement invert selection Modified: trunk/reactos/dll/win32/shell32/CDefView.cpp trunk/reactos/include/reactos/rosctrls.h Modified: trunk/reactos/dll/win32/shell32/CDefView.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/CDefView… ============================================================================== --- trunk/reactos/dll/win32/shell32/CDefView.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/shell32/CDefView.cpp [iso-8859-1] Sat Apr 4 18:51:28 2015 @@ -1554,9 +1554,10 @@ */ LRESULT CDefView::OnCommand(UINT uMsg, WPARAM wParam, LPARAM lParam, BOOL &bHandled) { - DWORD dwCmdID; - DWORD dwCmd; - HWND hwndCmd; + DWORD dwCmdID; + DWORD dwCmd; + HWND hwndCmd; + int nCount; dwCmdID = GET_WM_COMMAND_ID(wParam, lParam); dwCmd = GET_WM_COMMAND_CMD(wParam, lParam); @@ -1603,6 +1604,12 @@ case FCIDM_SHVIEW_SELECTALL: m_ListView.SetItemState(-1, LVIS_SELECTED, LVIS_SELECTED); + break; + + case FCIDM_SHVIEW_INVERTSELECTION: + nCount = m_ListView.GetItemCount(); + for (int i=0; i < nCount; i++) + m_ListView.SetItemState(i, m_ListView.GetItemState(i, LVIS_SELECTED) ? 0 : LVIS_SELECTED, LVIS_SELECTED); break; case FCIDM_SHVIEW_REFRESH: Modified: trunk/reactos/include/reactos/rosctrls.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/rosctrls.h… ============================================================================== --- trunk/reactos/include/reactos/rosctrls.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/rosctrls.h [iso-8859-1] Sat Apr 4 18:51:28 2015 @@ -165,6 +165,11 @@ spacing.cy = HIWORD(ret); } + UINT GetItemState(int i, UINT mask) + { + return SendMessage(LVM_GETITEMSTATE, i, (LPARAM)mask); + } + void SetItemState(int i, UINT state, UINT mask) { LV_ITEM item;

9 years, 8 months

[tkreuzer] 67051: [GDI32_APITEST] Add/fix some tests for CreateDIBPatternBrush, GetDIBits, PatBlt and SetWindowExtEx

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sat Apr 4 15:58:18 2015 New Revision: 67051 URL: http://svn.reactos.org/svn/reactos?rev=67051&view=rev Log: [GDI32_APITEST] Add/fix some tests for CreateDIBPatternBrush, GetDIBits, PatBlt and SetWindowExtEx Modified: trunk/rostests/apitests/gdi32/CreateDIBPatternBrush.c trunk/rostests/apitests/gdi32/GetDIBits.c trunk/rostests/apitests/gdi32/PatBlt.c trunk/rostests/apitests/gdi32/SetWindowExtEx.c Modified: trunk/rostests/apitests/gdi32/CreateDIBPatternBrush.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/CreateDIBP… ============================================================================== --- trunk/rostests/apitests/gdi32/CreateDIBPatternBrush.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/CreateDIBPatternBrush.c [iso-8859-1] Sat Apr 4 15:58:18 2015 @@ -9,6 +9,9 @@ #include <wingdi.h> #include "init.h" + +/* New color use parameter. See support.microsoft.com/kb/kbview/108497 */ +#define DIB_PAL_INDICES 2 void Test_CreateDIBPatternBrush() { @@ -30,6 +33,7 @@ }; HBRUSH hbr, hbrOld; HPALETTE hpalOld; + LOGBRUSH logbrush; SetLastError(0); ok_hdl(CreateDIBPatternBrushPt(NULL, 0), NULL); @@ -58,6 +62,13 @@ hbr = CreateDIBPatternBrushPt(&PackedDIB, DIB_PAL_COLORS); ok(hbr != 0, "CreateDIBPatternBrushPt failed, skipping tests.\n"); if (!hbr) return; + + /* Check the logbrush */ + ok(GetObject(hbr, sizeof(logbrush), &logbrush), "GetObject() failed\n"); + ok_int(logbrush.lbStyle, BS_DIBPATTERN); + ok_hex(logbrush.lbColor, 0); + ok(logbrush.lbHatch == (ULONG_PTR)&PackedDIB, + "invalid lbHatch. Got %p, expected %p\n", (PVOID)logbrush.lbHatch, &PackedDIB); /* Select the brush into the dc */ hbrOld = SelectObject(ghdcDIB32, hbr); @@ -100,10 +111,17 @@ PackedDIB.ajBuffer[2] = 1; PackedDIB.ajBuffer[3] = 0; - /* Create a DIB brush with unkdocumented iUsage == 2 */ - hbr = CreateDIBPatternBrushPt(&PackedDIB, 2); + /* Create a DIB brush with DIB_PAL_INDICES */ + hbr = CreateDIBPatternBrushPt(&PackedDIB, DIB_PAL_INDICES); ok(hbr != 0, "CreateSolidBrush failed, skipping tests.\n"); if (!hbr) return; + + /* Check the logbrush */ + ok(GetObject(hbr, sizeof(logbrush), &logbrush), "GetObject() failed\n"); + ok_int(logbrush.lbStyle, BS_DIBPATTERN); + ok_hex(logbrush.lbColor, 0); + ok(logbrush.lbHatch == (ULONG_PTR)&PackedDIB, + "invalid lbHatch. Got %p, expected %p\n", (PVOID)logbrush.lbHatch, &PackedDIB); /* Select the brush into the dc */ hbrOld = SelectObject(ghdcDIB32, hbr); @@ -155,8 +173,7 @@ /* Create a DIB brush with palette indices */ hbr = CreateDIBPatternBrushPt(&PackedDIB, DIB_PAL_COLORS); - ok(hbr != 0, "CreateDIBPatternBrushPt failed, skipping tests.\n"); - if (!hbr) return; + ok(hbr == 0, "CreateDIBPatternBrushPt should fail.\n"); } Modified: trunk/rostests/apitests/gdi32/GetDIBits.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/GetDIBits.… ============================================================================== --- trunk/rostests/apitests/gdi32/GetDIBits.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/GetDIBits.c [iso-8859-1] Sat Apr 4 15:58:18 2015 @@ -271,10 +271,24 @@ /* Test with different biBitCount set */ pbi->bmiHeader.biBitCount = 4; - pbi->bmiHeader.biSizeImage = 0; + pbi->bmiHeader.biSizeImage = 1; ok_int(GetDIBits(hdcScreen, hbmp, 0, 0, NULL, pbi, DIB_RGB_COLORS), 1); ok_int(pbi->bmiHeader.biSizeImage, 36); ok_int(pbi->bmiHeader.biBitCount, 4); + + /* Test with different biBitCount set */ + pbi->bmiHeader.biBitCount = 8; + pbi->bmiHeader.biSizeImage = 1000; + ok_int(GetDIBits(hdcScreen, hbmp, 0, 0, NULL, pbi, DIB_RGB_COLORS), 1); + ok_int(pbi->bmiHeader.biSizeImage, 60); + ok_int(pbi->bmiHeader.biBitCount, 8); + + /* Test with invalid biBitCount set */ + pbi->bmiHeader.biBitCount = 123; + pbi->bmiHeader.biSizeImage = -12; + ok_int(GetDIBits(hdcScreen, hbmp, 0, 0, NULL, pbi, DIB_RGB_COLORS), 0); + ok_int(pbi->bmiHeader.biSizeImage, -12); + ok_int(pbi->bmiHeader.biBitCount, 123); /* Set bitmap dimensions */ ok_int(SetBitmapDimensionEx(hbmp, 110, 220, NULL), 1); @@ -320,16 +334,21 @@ ok_int(pbi->bmiHeader.biCompression, 0); ok_int(pbi->bmiHeader.biSizeImage, 0); - /* Get the bitmap bits */ + /* Get the bitmap info */ pbi->bmiHeader.biSize = sizeof(BITMAPINFOHEADER); pbi->bmiHeader.biWidth = 4; pbi->bmiHeader.biHeight = 4; - pbi->bmiHeader.biPlanes = 1; + pbi->bmiHeader.biPlanes = 3; pbi->bmiHeader.biBitCount = 32; pbi->bmiHeader.biCompression = BI_RGB; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 1); + ok_int(pbi->bmiHeader.biSizeImage, 64); + ok_int(pbi->bmiHeader.biPlanes, 1); pbi->bmiHeader.biWidth = 0; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); + pbi->bmiHeader.biWidth = 2; + ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 1); + ok_int(pbi->bmiHeader.biSizeImage, 32); pbi->bmiHeader.biWidth = -3; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); pbi->bmiHeader.biWidth = 4; @@ -343,11 +362,23 @@ pbi->bmiHeader.biPlanes = 23; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 1); ok_int(pbi->bmiHeader.biPlanes, 1); + SetLastError(0xdeadbabe); + ok_int(GetDIBits((HDC)0xff00ff00, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); + ok_err(0x57); + SetLastError(0xdeadbabe); + ok_int(GetDIBits(hdcScreen, (HBITMAP)0xff00ff00, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); + ok_err(0xdeadbabe); + SetLastError(0xdeadbabe); + ok_int(GetDIBits((HDC)0xff00ff00, (HBITMAP)0xff00ff00, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); + ok_err(0x57); + SetLastError(0xdeadbabe); + ok_int(GetDIBits(NULL, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); + ok_err(0x57); + pbi->bmiHeader.biCompression = BI_JPEG; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); pbi->bmiHeader.biCompression = BI_PNG; ok_int(GetDIBits(hdcScreen, hbmp, 0, 5, NULL, pbi, DIB_RGB_COLORS), 0); - /* Get the bitmap bits */ pbi->bmiHeader.biSize = sizeof(BITMAPINFOHEADER); @@ -364,10 +395,22 @@ cjSizeImage = ((pbi->bmiHeader.biWidth * pbi->bmiHeader.biBitCount + 31) / 32) * 4 * pbi->bmiHeader.biHeight; pvBits = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, 512); ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 4); + ok_int(pbi->bmiHeader.biSize, sizeof(BITMAPINFOHEADER)); + ok_int(pbi->bmiHeader.biWidth, 4); + ok_int(pbi->bmiHeader.biHeight, 4); + ok_int(pbi->bmiHeader.biPlanes, 1); + ok_int(pbi->bmiHeader.biBitCount, 32); + ok_int(pbi->bmiHeader.biCompression, BI_RGB); + ok_int(pbi->bmiHeader.biSizeImage, 64); + ok_int(pbi->bmiHeader.biXPelsPerMeter, 0); + ok_int(pbi->bmiHeader.biYPelsPerMeter, 0); + ok_int(pbi->bmiHeader.biClrUsed, 0); + ok_int(pbi->bmiHeader.biClrImportant, 0); /* Set biBitCount to 0 */ pbi->bmiHeader.biBitCount = 0; ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 0); + ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, (PVOID)(LONG_PTR)-1, pbi, DIB_RGB_COLORS), 0); ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, NULL, pbi, DIB_RGB_COLORS), 1); ok_int(GetDIBits(NULL, hbmp, 0, 4, NULL, pbi, DIB_RGB_COLORS), 0); @@ -381,6 +424,10 @@ pbi->bmiHeader.biWidth = 3; ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 4); + /* Try invalid biBitCount */ + pbi->bmiHeader.biBitCount = 17; + ok_int(GetDIBits(hdcScreen, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 0); + /* Set only biBitCount and pjInit */ ZeroMemory(pbi, bisize); pbi->bmiHeader.biSize = sizeof(BITMAPINFOHEADER); @@ -405,19 +452,19 @@ pbi->bmiHeader.biSizeImage = 0; ok_int(GetDIBits(hdcScreen, hbmp, 0, 0, NULL, pbi, DIB_RGB_COLORS), 0); + /* Calculate bitmap size and allocate a buffer */ cjSizeImage = ((pbi->bmiHeader.biWidth * pbi->bmiHeader.biBitCount + 31) / 32) * 4 * pbi->bmiHeader.biHeight; + ok_int(cjSizeImage, 20); pvBits = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, cjSizeImage); + /* Test using a compatible DC */ hdcMem = CreateCompatibleDC(0); ok(hdcMem != 0, "CreateCompatibleDC failed, skipping tests\n"); if (hdcMem == NULL) return; - - // FIXME: broken - //ok(SelectObject(hdcMem, ghbmpDIB4) != 0, "Failed to select 4bpp DIB %p into DC %p\n", ghbmpDIB4, hdcMem);; - //ok_int(GetDIBits(hdcMem, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 0); - //ok_int(GetDIBits(hdcMem, ghbmpDIB4, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 3); - - + ok_int(GetDIBits(hdcMem, hbmp, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 0); + ok_int(GetDIBits(hdcMem, ghbmpDIB4, 0, 4, pvBits, pbi, DIB_RGB_COLORS), 4); + + HeapFree(GetProcessHeap(), 0, pvBits); DeleteDC(hdcMem); ReleaseDC(NULL, hdcScreen); } Modified: trunk/rostests/apitests/gdi32/PatBlt.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/PatBlt.c?r… ============================================================================== --- trunk/rostests/apitests/gdi32/PatBlt.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/PatBlt.c [iso-8859-1] Sat Apr 4 15:58:18 2015 @@ -22,6 +22,10 @@ /* Test a rop that contains only the operation index */ ret = PatBlt(hdcTarget, 0, 0, 1, 1, PATCOPY & 0x00FF0000); + ok_long(ret, 1); + + /* Test a rop that contains arbitrary values outside the operation index */ + ret = PatBlt(hdcTarget, 0, 0, 1, 1, (PATCOPY & 0x00FF0000) | 0xab00cdef); ok_long(ret, 1); /* Test an invalid rop */ Modified: trunk/rostests/apitests/gdi32/SetWindowExtEx.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/SetWindowE… ============================================================================== --- trunk/rostests/apitests/gdi32/SetWindowExtEx.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/SetWindowExtEx.c [iso-8859-1] Sat Apr 4 15:58:18 2015 @@ -96,7 +96,14 @@ //pDC_Attr = pEntry->UserData; //ASSERT(pDC_Attr); - /* Test setting it without changing the map mode (MM_TEXT) */ + /* Test setting 0 extents without changing the map mode (MM_TEXT) */ + ret = SetWindowExtEx(hDC, 0, 0, &WindowExt); + TEST(ret == 1); + TEST(WindowExt.cx == 1); + TEST(WindowExt.cy == 1); + + /* Test setting proper extents without changing the map mode (MM_TEXT) */ + WindowExt.cx = WindowExt.cy = 0; ret = SetWindowExtEx(hDC, 10, 20, &WindowExt); TEST(ret == 1); TEST(WindowExt.cx == 1);

9 years, 8 months

[tfaber] 67050: [USBEHCI] - Apply power to the right port in SetPortFeature

by tfaber＠svn.reactos.org

Author: tfaber Date: Sat Apr 4 15:50:38 2015 New Revision: 67050 URL: http://svn.reactos.org/svn/reactos?rev=67050&view=rev Log: [USBEHCI] - Apply power to the right port in SetPortFeature Modified: trunk/reactos/drivers/usb/usbehci/hardware.cpp Modified: trunk/reactos/drivers/usb/usbehci/hardware.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/usb/usbehci/hardwa… ============================================================================== --- trunk/reactos/drivers/usb/usbehci/hardware.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/usb/usbehci/hardware.cpp [iso-8859-1] Sat Apr 4 15:50:38 2015 @@ -1171,7 +1171,7 @@ // enable port power // Value = EHCI_READ_REGISTER_ULONG(EHCI_PORTSC + (4 * PortId)) | EHCI_PRT_POWER; - EHCI_WRITE_REGISTER_ULONG(EHCI_PORTSC, Value); + EHCI_WRITE_REGISTER_ULONG(EHCI_PORTSC + (4 * PortId), Value); // // delay is 20 ms

9 years, 8 months

[hbelusca] 67049: [CMD]: Support space and tabs as parameter separators in the DIR command.

by hbelusca＠svn.reactos.org

Author: hbelusca Date: Sat Apr 4 15:44:55 2015 New Revision: 67049 URL: http://svn.reactos.org/svn/reactos?rev=67049&view=rev Log: [CMD]: Support space and tabs as parameter separators in the DIR command. Modified: trunk/reactos/base/shell/cmd/dir.c Modified: trunk/reactos/base/shell/cmd/dir.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/dir.c?rev=6… ============================================================================== --- trunk/reactos/base/shell/cmd/dir.c [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/dir.c [iso-8859-1] Sat Apr 4 15:44:55 2015 @@ -163,7 +163,7 @@ { BOOL bBareFormat; /* Bare Format */ BOOL bTSeperator; /* Thousands seperator */ - BOOL bWideList; /* Wide list format */ + BOOL bWideList; /* Wide list format */ BOOL bWideListColSort; /* Wide list format but sorted by column */ BOOL bLowerCase; /* Uses lower case */ BOOL bNewLongList; /* New long list */ @@ -264,7 +264,7 @@ /* When a switch is expecting */ if (cCurSwitch == _T('/')) { - while (*Line == _T(' ')) + while (_istspace(*Line)) Line++; bNegative = (*Line == _T('-')); @@ -334,12 +334,12 @@ } else { - error_invalid_switch ((TCHAR)_totupper (*Line)); + error_invalid_switch ((TCHAR)_totupper(*Line)); return FALSE; } /* Make sure there's no extra characters at the end of the switch */ - if (Line[1] && Line[1] != _T('/') && Line[1] != _T(' ')) + if (Line[1] && Line[1] != _T('/') && !_istspace(Line[1])) { error_parameter_format(Line[1]); return FALSE; @@ -354,7 +354,7 @@ if (cCurChar == _T('/')) cCurSwitch = _T('/'); - else if (cCurChar == _T(' ')) + else if (_istspace(cCurChar)) /* do nothing */; else { @@ -363,7 +363,7 @@ bIntoQuotes = FALSE; while (*Line) { - if (!bIntoQuotes && (*Line == _T('/') || *Line == _T(' '))) + if (!bIntoQuotes && (*Line == _T('/') || _istspace(*Line))) break; bIntoQuotes ^= (*Line == _T('"')); Line++; @@ -394,7 +394,7 @@ /* We are waiting for switch parameters */ /* Check if there are no more switch parameters */ - if ((cCurChar == _T('/')) || ( cCurChar == _T(' '))) + if ((cCurChar == _T('/')) || _istspace(cCurChar)) { /* Wrong desicion path, reprocess current character */ cCurSwitch = _T(' '); @@ -403,7 +403,7 @@ /* Process parameter switch */ switch(cCurSwitch) { - case _T('A'): /* Switch parameters for /A (attributes filter) */ + case _T('A'): /* Switch parameters for /A (attributes filter) */ if (cCurChar == _T('-')) bPNegative = TRUE; else if (cCurUChar == _T('D')) @@ -452,7 +452,7 @@ return FALSE; } break; - case _T('T'): /* Switch parameters for /T (time field) */ + case _T('T'): /* Switch parameters for /T (time field) */ if (cCurUChar == _T('C')) lpFlags->stTimeField.eTimeField= TF_CREATIONDATE ; else if (cCurUChar == _T('A')) @@ -1155,7 +1155,7 @@ /* Calculate criteria */ switch(lpFlags->stOrderBy.eCriteria[i]) { - case ORDER_SIZE: /* Order by size /o:s */ + case ORDER_SIZE: /* Order by size /o:s */ /* concat the 32bit integers to a 64bit */ u64File1.LowPart = lpFile1->nFileSizeLow; u64File1.HighPart = lpFile1->nFileSizeHigh; @@ -1171,20 +1171,20 @@ iComp = 0; break; - case ORDER_DIRECTORY: /* Order by directory attribute /o:g */ + case ORDER_DIRECTORY: /* Order by directory attribute /o:g */ iComp = ((lpFile2->dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)- (lpFile1->dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)); break; - case ORDER_EXTENSION: /* Order by extension name /o:e */ + case ORDER_EXTENSION: /* Order by extension name /o:e */ iComp = _tcsicmp(getExt(lpFile1->cFileName),getExt(lpFile2->cFileName)); break; - case ORDER_NAME: /* Order by filename /o:n */ + case ORDER_NAME: /* Order by filename /o:n */ iComp = _tcsicmp(lpFile1->cFileName, lpFile2->cFileName); break; - case ORDER_TIME: /* Order by file's time /o:t */ + case ORDER_TIME: /* Order by file's time /o:t */ /* We compare files based on the time field selected by /t */ switch(lpFlags->stTimeField.eTimeField) { @@ -1333,7 +1333,7 @@ if (ptrStartNode == NULL) { WARN("DEBUG: Cannot allocate memory for ptrStartNode!\n"); - return 1; /* Error cannot allocate memory for 1st object */ + return 1; /* Error cannot allocate memory for 1st object */ } ptrNextNode = ptrStartNode;

9 years, 8 months

[hbelusca] 67048: [CMD]: Addendum to r67013: Check whether len > 0 before decrementing it in case we point to a newline. Fix some spurious crashes and should fix some other cmd_winetests.

by hbelusca＠svn.reactos.org

Author: hbelusca Date: Sat Apr 4 14:59:03 2015 New Revision: 67048 URL: http://svn.reactos.org/svn/reactos?rev=67048&view=rev Log: [CMD]: Addendum to r67013: Check whether len > 0 before decrementing it in case we point to a newline. Fix some spurious crashes and should fix some other cmd_winetests. Modified: trunk/reactos/base/shell/cmd/console.c Modified: trunk/reactos/base/shell/cmd/console.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/cmd/console.c?r… ============================================================================== --- trunk/reactos/base/shell/cmd/console.c [iso-8859-1] (original) +++ trunk/reactos/base/shell/cmd/console.c [iso-8859-1] Sat Apr 4 14:59:03 2015 @@ -179,7 +179,7 @@ /* Loop until we find a \r or \n character */ // FIXME: What about the pair \r\n ? p = str; - while (*(PWCHAR)p != L'\r' && *(PWCHAR)p != L'\n' && len > 0) + while (len > 0 && *(PWCHAR)p != L'\r' && *(PWCHAR)p != L'\n') { /* Advance one character */ p = (PVOID)((PWCHAR)p + 1); @@ -191,7 +191,7 @@ WriteFile(hOutput, str, dwNumBytes, &dwNumBytes, NULL); /* If we hit \r or \n ... */ - if (*(PWCHAR)p == L'\r' || *(PWCHAR)p == L'\n') + if (len > 0 && (*(PWCHAR)p == L'\r' || *(PWCHAR)p == L'\n')) { /* ... send a carriage-return + newline sequence and skip \r or \n */ WriteFile(hOutput, L"\r\n", 2 * sizeof(WCHAR), &dwNumBytes, NULL); @@ -229,7 +229,7 @@ /* Loop until we find a \r or \n character */ // FIXME: What about the pair \r\n ? p = str; - while (*(PCHAR)p != '\r' && *(PCHAR)p != '\n' && len > 0) + while (len > 0 && *(PCHAR)p != '\r' && *(PCHAR)p != '\n') { /* Advance one character */ p = (PVOID)((PCHAR)p + 1); @@ -241,7 +241,7 @@ WriteFile(hOutput, str, dwNumBytes, &dwNumBytes, NULL); /* If we hit \r or \n ... */ - if (*(PCHAR)p == '\r' || *(PCHAR)p == '\n') + if (len > 0 && (*(PCHAR)p == '\r' || *(PCHAR)p == '\n')) { /* ... send a carriage-return + newline sequence and skip \r or \n */ WriteFile(hOutput, "\r\n", 2, &dwNumBytes, NULL);

9 years, 8 months

[tkreuzer] 67047: [GDI32_APITEST] Add some more tests for MaskBlt

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sat Apr 4 14:58:17 2015 New Revision: 67047 URL: http://svn.reactos.org/svn/reactos?rev=67047&view=rev Log: [GDI32_APITEST] Add some more tests for MaskBlt Modified: trunk/rostests/apitests/gdi32/MaskBlt.c Modified: trunk/rostests/apitests/gdi32/MaskBlt.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/MaskBlt.c?… ============================================================================== --- trunk/rostests/apitests/gdi32/MaskBlt.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/MaskBlt.c [iso-8859-1] Sat Apr 4 14:58:17 2015 @@ -50,7 +50,7 @@ pjBitsMsk[0] = 0xF0; ret = MaskBlt(hdcDst, 0, 0, 8, 1, hdcSrc, 0, 0, hbmMsk, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); ok(ret == 1, "MaskBlt failed (%d)\n", ret); - ok (pjBitsDst[0] == 0xF0, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); + ok(pjBitsDst[0] == 0xF0, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); /* Do the masking (NOTSRCERASE / SRCINVERT) */ pjBitsDst[0] = 0xF0; // 11110000 @@ -59,7 +59,7 @@ ret = MaskBlt(hdcDst, 0, 0, 8, 1, hdcSrc, 0, 0, hbmMsk, 0, 0, MAKEROP4(NOTSRCERASE, SRCINVERT)); // 22 ok(ret == 1, "MaskBlt failed (%d)\n", ret); - ok (pjBitsDst[0] == 0x16, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); + ok(pjBitsDst[0] == 0x16, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); /* Do the masking (MERGEPAINT / DSxn) */ pjBitsDst[0] = 0xF0; @@ -67,7 +67,7 @@ pjBitsMsk[0] = 0xAA; ret = MaskBlt(hdcDst, 0, 0, 8, 1, hdcSrc, 0, 0, hbmMsk, 0, 0, MAKEROP4(MERGEPAINT, 0x990000)); ok(ret == 1, "MaskBlt failed (%d)\n", ret); - ok (pjBitsDst[0] == 0xE3, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); + ok(pjBitsDst[0] == 0xE3, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); /* Try a ROP that needs a mask with a NULL mask bitmap handle */ ret = MaskBlt(hdcDst, 0, 0, 8, 1, hdcSrc, 0, 0, NULL, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); @@ -90,6 +90,26 @@ /* Try (PATCOPY / NOOP) with a NULL source mask and bitmap */ ret = MaskBlt(hdcDst, 0, 0, 8, 1, NULL, 0, 0, NULL, 0, 0, MAKEROP4(PATCOPY, 0xAA0000)); ok(ret == 0, "MaskBlt should fail, but succeeded (%d)\n", ret); + + + /* Try with a mask that is smaller than the rect */ + DeleteObject(hbmMsk); + pbmi->bmiHeader.biWidth = 4; + hbmMsk = CreateDIBSection(hdcDst, pbmi, DIB_RGB_COLORS, (PVOID*)&pjBitsMsk, NULL, 0); + + /* Do the masking (SRCCOPY / NOOP) */ + pjBitsDst[0] = 0xAA; // 10101010 + pjBitsSrc[0] = 0xCC; // 11001100 + pjBitsMsk[0] = 0x33; // 00110011 + ret = MaskBlt(hdcDst, 0, 0, 5, 1, hdcSrc, 0, 0, hbmMsk, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 0, "MaskBlt should fail, but succeeded (%d)\n", ret); + ret = MaskBlt(hdcDst, 0, 0, 4, 1, hdcSrc, 0, 0, hbmMsk, 1, 0, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 0, "MaskBlt should fail, but succeeded (%d)\n", ret); + ret = MaskBlt(hdcDst, 0, 0, 4, 1, hdcSrc, 0, 0, hbmMsk, 0, 1, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 0, "MaskBlt should fail, but succeeded (%d)\n", ret); + ret = MaskBlt(hdcDst, 0, 0, 4, 1, hdcSrc, 0, 0, hbmMsk, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 1, "MaskBlt failed (%d)\n", ret); + ok(pjBitsDst[0] == 0x8A, "pjBitsDst[0] == 0x%x\n", pjBitsDst[0]); } @@ -181,15 +201,66 @@ ok (pulBitsDst[1] == 0xfabefef6, "pulBitsDst[0] == 0x%lx\n", pulBitsDst[1]); } +void Test_MaskBlt_Brush() +{ + HDC hdcDst, hdcSrc; + struct + { + BITMAPINFOHEADER bmiHeader; + ULONG aulColors[2]; + } bmiData = {{sizeof(BITMAPINFOHEADER), 16, 16, 1, 1, BI_RGB, 0, 10, 10, 2,0}, {0, 0xFFFFFF}}; + PBITMAPINFO pbmi = (PBITMAPINFO)&bmiData; + HBITMAP hbmDst, hbmSrc, hbmMsk; + PULONG pulBitsDst, pulBitsSrc, pulBitsMsk; + BOOL ret; + HBRUSH hbr; + + /* Create a dest dc and bitmap */ + hdcDst = CreateCompatibleDC(NULL); + hbmDst = CreateDIBSection(hdcDst, pbmi, DIB_RGB_COLORS, (PVOID*)&pulBitsDst, NULL, 0); + SelectObject(hdcDst, hbmDst); + + /* Create a source dc and bitmap */ + hdcSrc = CreateCompatibleDC(NULL); + hbmSrc = CreateDIBSection(hdcSrc, pbmi, DIB_RGB_COLORS, (PVOID*)&pulBitsSrc, NULL, 0); + SelectObject(hdcSrc, hbmSrc); + + hbr = CreateHatchBrush(HS_CROSS, 0); + ok(hbr != 0, "failed to create brush\n"); + ok(SelectObject(hdcDst, hbr) != 0, "failed to select brush\n"); + + /* Do the masking (SRCCOPY / NOOP) */ + pulBitsDst[0] = 0x00000000; + pulBitsSrc[0] = 0xFFFFFFFF; + ret = MaskBlt(hdcDst, 0, 0, 8, 1, hdcSrc, 0, 0, NULL, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 1, "MaskBlt failed (%d)\n", ret); + ok(pulBitsDst[0] == 0, "pulBitsDst[0] == 0x%lx\n", pulBitsDst[0]); + + /* Create a 1 bpp pattern brush */ + pbmi->bmiHeader.biWidth = 8; + hbmMsk = CreateDIBSection(hdcDst, pbmi, DIB_RGB_COLORS, (PVOID*)&pulBitsMsk, NULL, 0); + ok(hbmMsk != 0, "CreateDIBSection failed\n"); + hbr = CreatePatternBrush(hbmMsk); + ok(hbr != 0, "CreatePatternBrush failed\n"); + ok(SelectObject(hdcDst, hbr) != 0, "failed to select brush\n"); + + /* Do the masking (SRCCOPY / NOOP) */ + pulBitsDst[0] = 0x00000000; + pulBitsSrc[0] = 0xFFFFFFFF; + pulBitsMsk[0] = 0xCCAAFF00; + ret = MaskBlt(hdcDst, 0, 0, 16, 1, hdcSrc, 0, 0, NULL, 0, 0, MAKEROP4(SRCCOPY, 0xAA0000)); + ok(ret == 1, "MaskBlt failed (%d)\n", ret); + ok(pulBitsDst[0] == 0, "pulBitsDst[0] == 0x%lx\n", pulBitsDst[0]); + +} + START_TEST(MaskBlt) { InitStuff(); Test_MaskBlt_1bpp(); - switch (GetDeviceCaps(GetDC(NULL), BITSPIXEL)) - { - case 16: Test_MaskBlt_16bpp(); break; - case 32: Test_MaskBlt_32bpp(); break; - } - -} - + Test_MaskBlt_16bpp(); + Test_MaskBlt_32bpp(); + Test_MaskBlt_Brush(); + +} +

9 years, 8 months

[tkreuzer] 67046: [GDI32_APITEST] Fix some broken tests or disable them.

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sat Apr 4 14:51:20 2015 New Revision: 67046 URL: http://svn.reactos.org/svn/reactos?rev=67046&view=rev Log: [GDI32_APITEST] Fix some broken tests or disable them. Modified: trunk/rostests/apitests/gdi32/GetClipBox.c trunk/rostests/apitests/gdi32/GetRandomRgn.c trunk/rostests/apitests/gdi32/OffsetClipRgn.c Modified: trunk/rostests/apitests/gdi32/GetClipBox.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/GetClipBox… ============================================================================== --- trunk/rostests/apitests/gdi32/GetClipBox.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/GetClipBox.c [iso-8859-1] Sat Apr 4 14:51:20 2015 @@ -40,7 +40,7 @@ SetLastError(ERROR_SUCCESS); ret = GetClipBox((HDC)0x12345, &rect); ok(ret == ERROR, "Expected ERROR, got %d\n", ret); - ok(GetLastError() == 0, "Expected 0, got %ld\n", GetLastError()); + ok((GetLastError() == 0) || (GetLastError() == ERROR_INVALID_HANDLE), "Expected 0, got %ld\n", GetLastError()); //ret = GetClipBox(hdc, &rect); //ok_int(ret, SIMPLEREGION); Modified: trunk/rostests/apitests/gdi32/GetRandomRgn.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/GetRandomR… ============================================================================== --- trunk/rostests/apitests/gdi32/GetRandomRgn.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/GetRandomRgn.c [iso-8859-1] Sat Apr 4 14:51:20 2015 @@ -74,12 +74,12 @@ ret = GetRandomRgn(hdc, hrgn, 0); ok_int(ret, 0); ok_long(GetLastError(), 0xbadbad00); - +#if 0 // this is vista+ SetLastError(0xbadbad00); ret = GetRandomRgn(hdc, hrgn, 5); ok_int(ret, 1); ok_long(GetLastError(), 0xbadbad00); - +#endif SetLastError(0xbadbad00); ret = GetRandomRgn(hdc, hrgn, 6); ok_int(ret, 0); @@ -240,14 +240,100 @@ ok_int(ret, 1); GetRgnBox(hrgn1, &rect); DPtoLP(ghdcWindow, (LPPOINT)&rect, 2); +#if 0 // FIXME: this needs calculation ok_long(rect.left, 104); ok_long(rect.top, 124); ok_long(rect.right, 209); ok_long(rect.bottom, 196); +#endif MoveWindow(ghwnd, 200, 400, 200, 200, 0); ret = GetRandomRgn(ghdcWindow, hrgn1, SYSRGN); + ok_int(ret, 1); + GetRgnBox(hrgn1, &rect2); + DPtoLP(ghdcWindow, (LPPOINT)&rect2, 2); +#if 0 // FIXME: this needs calculation + ok_long(rect2.left, rect.left + 100); + ok_long(rect2.top, rect.top + 300); + ok_long(rect2.right, rect.right + 200 - 13); + ok_long(rect2.bottom, rect.bottom + 400); +#endif + + DeleteObject(hrgn1); + DeleteDC(hdc); + +} + +void Test_GetRandomRgn_RGN5() +{ + HDC hdc; + HRGN hrgn1, hrgn2; + INT ret; + RECT rect, rect2; + HBITMAP hbmp; + DBG_UNREFERENCED_LOCAL_VARIABLE(hrgn2); + DBG_UNREFERENCED_LOCAL_VARIABLE(rect2); + + hrgn1 = CreateRectRgn(11, 17, 23, 42); + if (!hrgn1) + { + printf("Coun't create a region\n"); + return; + } + + hdc = CreateCompatibleDC(0); + if (!hdc) + { + printf("Coun't create a dc\n"); + return; + } +#if 0 // this is vista+ + ret = GetRandomRgn(hdc, hrgn1, RGN5); + ok_int(ret, 1); + GetRgnBox(hrgn1, &rect); + ok_long(rect.left, 0); + ok_long(rect.top, 0); + ok_long(rect.right, 1); + ok_long(rect.bottom, 1); + + hrgn2 = CreateRectRgn(1, 2, 3, 4); + SelectClipRgn(hdc, hrgn2); + DeleteObject(hrgn2); + ret = GetRandomRgn(hdc, hrgn1, RGN5); + ok_int(ret, 1); + GetRgnBox(hrgn1, &rect); + ok_long(rect.left, 0); + ok_long(rect.top, 0); + ok_long(rect.right, 1); + ok_long(rect.bottom, 1); +#endif + + hbmp = CreateCompatibleBitmap(hdc, 4, 7); + SelectObject(hdc, hbmp); + ret = GetRandomRgn(hdc, hrgn1, SYSRGN); + ok_int(ret, 1); + GetRgnBox(hrgn1, &rect); + ok_long(rect.left, 0); + ok_long(rect.top, 0); + ok_long(rect.right, 4); + ok_long(rect.bottom, 7); + DeleteObject(hbmp); + +#if 0 // this is vista+ + MoveWindow(ghwnd, 100, 100, 100, 100, 0); + ret = GetRandomRgn(ghdcWindow, hrgn1, RGN5); + ok_int(ret, 1); + GetRgnBox(hrgn1, &rect); + DPtoLP(ghdcWindow, (LPPOINT)&rect, 2); + ok_long(rect.left, 104); + ok_long(rect.top, 124); + ok_long(rect.right, 209); + ok_long(rect.bottom, 196); + + MoveWindow(ghwnd, 200, 400, 200, 200, 0); + + ret = GetRandomRgn(ghdcWindow, hrgn1, RGN5); ok_int(ret, 1); GetRgnBox(hrgn1, &rect2); DPtoLP(ghdcWindow, (LPPOINT)&rect2, 2); @@ -255,86 +341,7 @@ ok_long(rect2.top, rect.top + 300); ok_long(rect2.right, rect.right + 200 - 13); ok_long(rect2.bottom, rect.bottom + 400); - - - DeleteObject(hrgn1); - DeleteDC(hdc); - -} - -void Test_GetRandomRgn_RGN5() -{ - HDC hdc; - HRGN hrgn1, hrgn2; - INT ret; - RECT rect, rect2; - HBITMAP hbmp; - - hrgn1 = CreateRectRgn(11, 17, 23, 42); - if (!hrgn1) - { - printf("Coun't create a region\n"); - return; - } - - hdc = CreateCompatibleDC(0); - if (!hdc) - { - printf("Coun't create a dc\n"); - return; - } - - ret = GetRandomRgn(hdc, hrgn1, RGN5); - ok_int(ret, 1); - GetRgnBox(hrgn1, &rect); - ok_long(rect.left, 0); - ok_long(rect.top, 0); - ok_long(rect.right, 1); - ok_long(rect.bottom, 1); - - hrgn2 = CreateRectRgn(1, 2, 3, 4); - SelectClipRgn(hdc, hrgn2); - DeleteObject(hrgn2); - ret = GetRandomRgn(hdc, hrgn1, RGN5); - ok_int(ret, 1); - GetRgnBox(hrgn1, &rect); - ok_long(rect.left, 0); - ok_long(rect.top, 0); - ok_long(rect.right, 1); - ok_long(rect.bottom, 1); - - hbmp = CreateCompatibleBitmap(hdc, 4, 7); - SelectObject(hdc, hbmp); - ret = GetRandomRgn(hdc, hrgn1, SYSRGN); - ok_int(ret, 1); - GetRgnBox(hrgn1, &rect); - ok_long(rect.left, 0); - ok_long(rect.top, 0); - ok_long(rect.right, 4); - ok_long(rect.bottom, 7); - DeleteObject(hbmp); - - MoveWindow(ghwnd, 100, 100, 100, 100, 0); - ret = GetRandomRgn(ghdcWindow, hrgn1, RGN5); - ok_int(ret, 1); - GetRgnBox(hrgn1, &rect); - DPtoLP(ghdcWindow, (LPPOINT)&rect, 2); - ok_long(rect.left, 104); - ok_long(rect.top, 124); - ok_long(rect.right, 209); - ok_long(rect.bottom, 196); - - MoveWindow(ghwnd, 200, 400, 200, 200, 0); - - ret = GetRandomRgn(ghdcWindow, hrgn1, RGN5); - ok_int(ret, 1); - GetRgnBox(hrgn1, &rect2); - DPtoLP(ghdcWindow, (LPPOINT)&rect2, 2); - ok_long(rect2.left, rect.left + 100); - ok_long(rect2.top, rect.top + 300); - ok_long(rect2.right, rect.right + 200 - 13); - ok_long(rect2.bottom, rect.bottom + 400); - +#endif DeleteObject(hrgn1); DeleteDC(hdc); Modified: trunk/rostests/apitests/gdi32/OffsetClipRgn.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/OffsetClip… ============================================================================== --- trunk/rostests/apitests/gdi32/OffsetClipRgn.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/OffsetClipRgn.c [iso-8859-1] Sat Apr 4 14:51:20 2015 @@ -30,7 +30,7 @@ /* Test invalid DC */ SetLastError(0x12345); ok_int(OffsetClipRgn((HDC)(ULONG_PTR)0x12345, 0, 0), ERROR); - ok_int(GetLastError(), 0x12345); + ok((GetLastError() == 0x12345) || (GetLastError() == ERROR_INVALID_HANDLE), "Expected 0x12345 or ERROR_INVALID_HANDLE, got %ld\n", GetLastError()); SetLastError(0x12345); /* Test without a clip region set */

9 years, 8 months

[tkreuzer] 67045: [GDI32_APITEST] Improve/fix tests for CreateBitmap.

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sat Apr 4 14:38:04 2015 New Revision: 67045 URL: http://svn.reactos.org/svn/reactos?rev=67045&view=rev Log: [GDI32_APITEST] Improve/fix tests for CreateBitmap. Modified: trunk/rostests/apitests/gdi32/CreateBitmap.c Modified: trunk/rostests/apitests/gdi32/CreateBitmap.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/gdi32/CreateBitm… ============================================================================== --- trunk/rostests/apitests/gdi32/CreateBitmap.c [iso-8859-1] (original) +++ trunk/rostests/apitests/gdi32/CreateBitmap.c [iso-8859-1] Sat Apr 4 14:38:04 2015 @@ -31,38 +31,38 @@ SetLastError(0); hbmp = CreateBitmap(1, -1, 1, 0, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); SetLastError(0); hbmp = CreateBitmap(-1, 1, 1, 0, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); SetLastError(0); hbmp = CreateBitmap(-1, 1, 1, 1, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); SetLastError(0); hbmp = CreateBitmap(1, -1, 1, 1, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); /* Check if an overflow in cPlanes * cBitsPixel is handled */ SetLastError(0); hbmp = CreateBitmap(1, 1, 2, 0x80000004, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); /* Check for maximum width */ hbmp = CreateBitmap(0x7FFFFFF, 1, 1, 1, NULL); - ok(hbmp != 0, "\n"); + ok(hbmp != 0, "CreateBitmap failed\n"); DeleteObject(hbmp); SetLastError(0); hbmp = CreateBitmap(0x8000000, 1, 1, 1, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); /* Check for maximum height */ @@ -72,24 +72,46 @@ SetLastError(0); hbmp = CreateBitmap(1, 0x1FFFFFFF, 1, 1, NULL); - ok(hbmp == 0, "\n"); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(0); - /* Check for overflow in width * height */ - hbmp = CreateBitmap(0x2000, 0x4000, 1, 1, NULL); //128 MB.Should work. - ok(hbmp != 0, "\n"); - DeleteObject(hbmp); - - /* XP doesn't allow to create bitmaps larger than 128 Mb */ - SetLastError(0); - hbmp = CreateBitmap(0x2000, 0x4001, 1, 1, NULL);//More than 128MB.Should fail. - ok(hbmp == 0, "\n"); - ok_err(0); - - /* Check huge allocation */ - SetLastError(0); - hbmp = CreateBitmap(0x2000, 0x20000, 32, 1, NULL); - ok(hbmp == 0, "\n"); + SetLastError(0); + hbmp = CreateBitmap(1, -1, 1, 1, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); + ok_err(ERROR_INVALID_PARAMETER); + + /* Test huge allocation (256 GB) */ + SetLastError(0); + hbmp = CreateBitmap(0x40000, 0x40000, 32, 1, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); + ok_err(ERROR_INVALID_PARAMETER); + + /* Test planes / bpp */ + hbmp = CreateBitmap(10, 10, 32, 1, NULL); + ok(hbmp != 0, "CreateBitmap failed\n"); + DeleteObject(hbmp); + hbmp = CreateBitmap(10, 10, 5, 5, NULL); + ok(hbmp != 0, "CreateBitmap failed\n"); + DeleteObject(hbmp); + + SetLastError(0); + hbmp = CreateBitmap(10, 10, 33, 1, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); + ok_err(ERROR_INVALID_PARAMETER); + + SetLastError(0); + hbmp = CreateBitmap(10, 10, 1, 33, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); + ok_err(ERROR_INVALID_PARAMETER); + + SetLastError(0); + hbmp = CreateBitmap(10, 10, 6, 6, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); + ok_err(ERROR_INVALID_PARAMETER); + + SetLastError(0); + hbmp = CreateBitmap(10, 10, 8, 8, NULL); + ok(hbmp == 0, "CreateBitmap should fail\n"); ok_err(ERROR_INVALID_PARAMETER); }

9 years, 8 months

← Newer
1
...
41
42
43
44
45
46
47
48
49
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 April 2015