Ros-diffs December 2012

ros-diffs@reactos.org

12 participants
287 discussions

[tkreuzer] 57922: [PSDK/WIN32K] - Add suppress.h - Use symbolic error definitions with _PRAGMA_WARNING_SUPPRESS
by tkreuzer＠svn.reactos.org 16 Dec '12

16 Dec '12

Author: tkreuzer Date: Sun Dec 16 11:32:56 2012 New Revision: 57922 URL: http://svn.reactos.org/svn/reactos?rev=57922&view=rev Log: [PSDK/WIN32K] - Add suppress.h - Use symbolic error definitions with _PRAGMA_WARNING_SUPPRESS Added: trunk/reactos/include/psdk/suppress.h (with props) Modified: trunk/reactos/include/crt/mingw32/intrin.h trunk/reactos/include/crt/msc/intrin.h trunk/reactos/include/reactos/probe.h trunk/reactos/win32ss/gdi/ntgdi/path.c trunk/reactos/win32ss/gdi/ntgdi/region.c Modified: trunk/reactos/include/crt/mingw32/intrin.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/crt/mingw32/intrin… ============================================================================== --- trunk/reactos/include/crt/mingw32/intrin.h [iso-8859-1] (original) +++ trunk/reactos/include/crt/mingw32/intrin.h [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -36,7 +36,6 @@ #define __ATTRIBUTE_ARTIFICIAL __attribute__((artificial)) #endif -#define _PRAGMA_WARNING_SUPPRESS(x) /* Only for MSVC */ #define __INTRIN_INLINE extern __inline__ __attribute__((__always_inline__,__gnu_inline__)) __ATTRIBUTE_ARTIFICIAL #ifndef _SIZE_T_DEFINED Modified: trunk/reactos/include/crt/msc/intrin.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/crt/msc/intrin.h?r… ============================================================================== --- trunk/reactos/include/crt/msc/intrin.h [iso-8859-1] (original) +++ trunk/reactos/include/crt/msc/intrin.h [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -4,9 +4,6 @@ #ifdef __cplusplus extern "C" { #endif - -/** Pragma support **/ -#define _PRAGMA_WARNING_SUPPRESS(x) __pragma(warning(suppress: x)) /*** Stack frame juggling ***/ void * _ReturnAddress(void); Added: trunk/reactos/include/psdk/suppress.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/suppress.h?re… ============================================================================== --- trunk/reactos/include/psdk/suppress.h (added) +++ trunk/reactos/include/psdk/suppress.h [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -1,0 +1,827 @@ +/* + * suppress.h + * + * Symbolic macro definitions for compiler warnings + * + * This file is part of the ReactOS PSDK package. + * + * Contributors: + * Timo Kreuzer (timo.kreuzer(a)reactos.org) + * + * THIS SOFTWARE IS NOT COPYRIGHTED + * + * This source code is offered for use in the public domain. You may + * use, modify or distribute it freely. + * + * This code is distributed in the hope that it will be useful but + * WITHOUT ANY WARRANTY. ALL WARRANTIES, EXPRESS OR IMPLIED ARE HEREBY + * DISCLAIMED. This includes but is not limited to warranties of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. + * + */ + +#pragma once + +#ifdef _MSC_VER +#define _PRAGMA_WARNING_SUPPRESS(x) __pragma(warning(suppress: x)) +#else +#define _PRAGMA_WARNING_SUPPRESS(x) /* Only for MSVC */ +#endif + +#define __WARNING_USING_UNINIT_VAR 6001 +#define __WARNING_DEREF_NULL_PTR 6011 +#define __WARNING_MEMORY_LEAK 6014 +#define __WARNING_USING_TAINTED_DATA 6029 +#define __WARNING_RETVAL_IGNORED_FUNC_COULD_FAIL 6031 +#define __WARNING_MISSING_ZERO_TERMINATION1 6053 +#define __WARNING_MISSING_ZERO_TERMINATION2 6054 +#define __WARNING_CHAR_BYTE_SIZE_MISMATCH 6057 +#define __WARNING_BAD_CONCATENATION 6059 +#define __WARNING_MISSING_STRING_ARGUMENT_TO_FORMAT_FUNCTION 6063 +#define __WARNING_MISSING_INTEGER_ARGUMENT_TO_FORMAT_FUNCTION 6064 +#define __WARNING_NON_POINTER_ARGUMENT_TO_FORMAT_FUNCTION 6066 +#define __WARNING_NON_STRING_ARGUMENT_TO_FORMAT_FUNCTION 6067 +#define __WARNING_INDEX_EXCEEDS_MAX_NONSTACK 6200 +#define __WARNING_INDEX_EXCEEDS_MAX 6201 +#define __WARNING_BUFFER_OVERRUN 6202 +#define __WARNING_BUFFER_OVERRUN_NONSTACK 6203 +#define __WARNING_UNVALIDATED_PARAM 6204 +#define __WARNING_SIZEOF_CHAR_BYTE_SIZE_MISMATCH 6209 +#define __WARNING_MEMORY_LEAK_EXCEPTION 6211 +#define __WARNING_CAST_HRESULT_TO_BOOL 6214 +#define __WARNING_CAST_BOOL_TO_HRESULT 6215 +#define __WARNING_COMPILER_INSERTED_CAST_BOOL_TO_HRESULT 6216 +#define __WARNING_TESTING_HRESULT_WITH_NOT 6217 +#define __WARNING_COMPARING_HRESULT_TO_ONE 6219 +#define __WARNING_COMPARING_HRESULT_TO_MINUS_ONE 6220 +#define __WARNING_COMPARING_HRESULT_TO_INT 6221 +#define __WARNING_ASSIGNING_ONE_TO_HRESULT 6225 +#define __WARNING_ASSIGNING_MINUS_ONE_TO_HRESULT 6226 +#define __WARNING_USING_HRESULT_IN_BOOLEAN_CONTEXT 6230 +#define __WARNING_NONZEROLOGICALOR 6235 +#define __WARNING_LOGICALORNONZERO 6236 +#define __WARNING_ZEROLOGICALANDLOSINGSIDEEFFECTS 6237 +#define __WARNING_NONZEROLOGICALAND 6239 +#define __WARNING_LOGICALANDNONZERO 6240 +#define __WARNING_LOCALUNWINDFORCED 6242 +#define __WARNING_LOCALDECLHIDESGLOBAL 6244 +#define __WARNING_LOCALDECLHIDESLOCAL 6246 +#define __WARNING_CREATINGNULLDACL 6248 +#define __WARNING_WIN32UNRELEASEDVADS 6250 +#define __WARNING_UNPROTECTEDUSEOFALLOCA 6255 +#define __WARNING_USINGTERMINATETHREAD 6258 +#define __WARNING_DEADCODEINBITORLIMITEDSWITCH 6259 +#define __WARNING_USEOFBYTEAREA 6260 +#define __WARNING_EXCESSIVESTACKUSAGE 6262 +#define __WARNING_USINGALLOCAINLOOP 6263 +#define __WARNING_STRING_LITERAL_TO_NONCONST_CAST 6267 +#define __WARNING_MISPARENTHESIZED_CASTS 6268 +#define __WARNING_POINTER_DEREF_DISCARDED 6269 +#define __WARNING_MISSING_FLOAT_ARGUMENT_TO_FORMAT_FUNCTION 6270 +#define __WARNING_EXTRA_ARGUMENT_TO_FORMAT_FUNCTION 6271 +#define __WARNING_NON_FLOAT_ARGUMENT_TO_FORMAT_FUNCTION 6272 +#define __WARNING_NON_INTEGER_ARGUMENT_TO_FORMAT_FUNCTION 6273 +#define __WARNING_NON_CHAR_ARGUMENT_TO_FORMAT_FUNCTION 6274 +#define __WARNING_CHAR_TO_WCHAR_CAST 6276 +#define __WARNING_CREATEPROCESS_ESCAPE 6277 +#define __WARNING_ARRAY_NEW_DELETE_MISMATCH 6278 +#define __WARNING_NEW_ARRAY_DELETE_MISMATCH 6279 +#define __WARNING_MEMORY_ALLOCATION_MISMATCH 6280 +#define __WARNING_BITWISERELATIONPRECEDENCEERROR 6281 +#define __WARNING_ASSIGNMENTREPLACESTEST 6282 +#define __WARNING_PRIMITIVE_ARRAY_NEW_DELETE_MISMATCH 6283 +#define __WARNING_OBJECT_AS_STRING_ARGUMENT_TO_FORMAT_FUNCTION 6284 +#define __WARNING_LOGICALOROFCONSTANTS 6285 +#define __WARNING_NONZEROLOGICALORLOSINGSIDEEFFECTS 6286 +#define __WARNING_REDUNDANTTEST 6287 +#define __WARNING_MUTUALINCLUSIONOVERANDISFALSE 6288 +#define __WARNING_MUTUALEXCLUSIONOVERORISTRUE 6289 +#define __WARNING_LOGICALNOTBITWISEAND 6290 +#define __WARNING_LOGICALNOTBITWISEOR 6291 +#define __WARNING_LOOP_COUNTS_UP_FROM_MAX 6292 +#define __WARNING_LOOP_INDEX_GOES_NEGATIVE 6293 +#define __WARNING_LOOP_BODY_NEVER_EXECUTED 6294 +#define __WARNING_INFINITE_LOOP 6295 +#define __WARNING_LOOP_ONLY_EXECUTED_ONCE 6296 +#define __WARNING_RESULTOFSHIFTCASTTOLARGERSIZE 6297 +#define __WARNING_CONST_STRING_TO_WRITABLE_STRING 6298 +#define __WARNING_BITFIELD_TO_BOOL_COMPARISON 6299 +#define __WARNING_CHAR_WCHAR_ARGUMENT_TO_FORMAT_FUNCTION 6302 +#define __WARNING_WCHAR_CHAR_ARGUMENT_TO_FORMAT_FUNCTION 6303 +#define __WARNING_SIZEOF_COUNTOF_MISMATCH 6305 +#define __WARNING_INCORRECT_VARARG_FUNCTIONCALL 6306 +#define __WARNING_WCHAR_TO_BSTR 6307 +#define __WARNING_REALLOCLEAK 6308 +#define __WARNING_PASSING_FUNCTION_UNEXPECTED_NULL 6309 +#define __WARNING_ILLEGALEXCEPTEXPRCONST 6310 +#define __WARNING_EXCEPTIONCONTINUEEXECUTION 6312 +#define __WARNING_BITANDVSZEROVALUEDFLAG 6313 +#define __WARNING_BITORVSQUESTION 6314 +#define __WARNING_BITORVSBITAND 6315 +#define __WARNING_INAPPROPRIATEUSEOFBITOR 6316 +#define __WARNING_NOTNOTCOMPLEMENT 6317 +#define __WARNING_EXCEPTIONCONTINUESEARCH 6318 +#define __WARNING_IGNOREDBYCOMMA 6319 +#define __WARNING_EXCEPTIONEXECUTEHANDLER 6320 +#define __WARNING_EXCEPT_BLOCK_EMPTY 6322 +#define __WARNING_ARITH_OP_ON_BOOL 6323 +#define __WARNING_STRCPY_INSTEAD_OF_STRCMP 6324 +#define __WARNING_CONST_CONST_COMP 6326 +#define __WARNING_DEST_BUFFER_INSUFFICIENT 6327 +#define __WARNING_POTENTIAL_INCORRECT_RETVAL_CHECK 6329 +#define __WARNING_POTENTIAL_ARGUMENT_TYPE_MISMATCH 6330 +#define __WARNING_VIRTUALFREEINVALIDPARAM1 6331 +#define __WARNING_VIRTUALFREEINVALIDPARAM2 6332 +#define __WARNING_VIRTUALFREEINVALIDPARAM3 6333 +#define __WARNING_SIZEOFEXPR 6334 +#define __WARNING_LEAKING_PROCESS_HANDLE 6335 +#define __WARNING_QUESTIONPRECEDENCE 6336 +#define __WARNING_SHUTDOWN_API 6381 +#define __WARNING_ELEMENTS_TO_BYTES 6383 +#define __WARNING_DIVIDING_SIZEOF_POINTER 6384 +#define __WARNING_READ_OVERRUN 6385 +#define __WARNING_WRITE_OVERRUN 6386 +#define __WARNING_INVALID_PARAM_VALUE_1 6387 +#define __WARNING_INVALID_PARAM_VALUE_2 6388 +#define __WARNING_BOOL_VARIANT_BOOL_COMPARISON 6392 +#define __WARNING_COMPARE64BIT 6393 +#define __WARNING_MS03_026_LHS 6394 +#define __WARNING_LOCALE_DEPENDENT_CONSTANT_STRING_COMPARISON 6400 +#define __WARNING_DEFAULT_LOCALE_CONSTANT_STRING_COMPARISON 6401 +#define __WARNING_INVALID_ATTRIBUTE_PROPERTY 6500 +#define __WARNING_CONFLICTING_ATTRIBUTE_PROPERTY_VALUES 6501 +#define __WARNING_REFERENCES_CANT_BE_NULL 6503 +#define __WARNING_NULL_ON_NON_POINTER 6504 +#define __WARNING_MUSTCHECK_ON_VOID 6505 +#define __WARNING_BUFFER_SIZE_ON_NON_POINTER_OR_ARRAY 6506 +#define __WARNING_NULL_MISMATCH_AT_DEREF_0 6507 +#define __WARNING_WRITE_ACCESS_ON_CONST 6508 +#define __WARNING_RETURN_USED_ON_PRECONDITION 6509 +#define __WARNING_NULLTERMINATED_ON_NON_POINTER 6510 +#define __WARNING_MUSTCHECK_MAYBE 6511 +#define __WARNING_NULL_CONFLICTS_WITH_NOT_VALID 6512 +#define __WARNING_ELEMENT_SIZE_WITHOUT_BUFFER_SIZE 6513 +#define __WARNING_BUFFER_SIZE_EXCEEDS_ARRAY_SIZE 6514 +#define __WARNING_BUFFER_SIZE_ON_NON_POINTER 6515 +#define __WARNING_NO_PROPERTIES_ON_ATTRIBUTE 6516 +#define __WARNING_VALID_SIZE_ON_NON_READABLE_BUFFER 6517 +#define __WARNING_WRITABLE_SIZE_ON_NON_WRITABLE_BUFFER 6518 +#define __WARNING_NEEDSRELEASE_MAYBE 6519 +#define __WARNING_INVALID_SIZE_STRING_DEREF 6521 +#define __WARNING_INVALID_SIZE_STRING_TYPE 6522 +#define __WARNING_INVALID_SIZE_STRING_PARAM 6523 +#define __WARNING_INVALID_SIZE_STRING_UNREACHABLE_LOCATION 6525 +#define __WARNING_INVALID_SIZE_STRING_BUFFER_TYPE 6526 +#define __WARNING_NEEDSRELEASE_ON_VOID 6527 +#define __WARNING_UNRECOGNIZED_FORMAT_STRING_STYLE 6530 +#define __WARNING_BUFFERSIZE_INFEASIBLE 6535 +#define __WARNING_OVERRIDING_OLD_MODEL 6540 +#define __WARNING_LIB_BADYESNO 6701 +#define __WARNING_LIB_BADSTRING 6702 +#define __WARNING_LIB_BADNUMBER 6703 +#define __WARNING_LIB_EXP_MISC 6704 +#define __WARNING_LIB_ANNOTE_ARITY 6705 +#define __WARNING_LIB_ANNOTE_MISC 6706 +#define __WARNING_LIB_MODEL_MISC 6707 +#define __WARNING_MANAGEDCODENOTSUPPORTED 6990 +#define __WARNING_ATTRIBUTESNOTSUPPORTED 6991 +#define __WARNING_NATIVEEVENTSYNTAXNOTSUPPORTED 6992 +#define __WARNING_OMPNOTSUPPORTED 6993 +#define __WARNING_PCHNOTFOUND 6994 +#define __WARNING__SAVETOXMLFAILED 6995 +#define __WARNING_DUMMY_IO_WARNING1 22010 +#define __WARNING_ALLOC_SIZE_OVERFLOW 22011 +#define __WARNING_ALLOC_SIZE_WRONG_OVERFLOW_CHECK 22012 +#define __WARNING_INDEX_WRONG_OVERFLOW_CHECK 22013 +#define __WARNING_ALLOC_SIZE_NEGATIVE 22014 +#define __WARNING_INDEX_NEGATIVE 22015 +#define __WARNING_LOOP_INDEX_WRONG_OVERFLOW_CHECK 22016 +#define __WARNING_ALLOC_SIZE_UNDERFLOW 22017 +#define __WARNING_INDEX_UNDERFLOW 22018 +#define __WARNING_LOOP_INDEX_UNDERFLOW 22019 +#define __WARNING_SIGNED_UNSIGNED_COMPARISON 22020 +#define __WARNING_DIFF_SIZE_SIGNED_UNSIGNED_COMPARISON 22021 +#define __WARNING_ALLOC_SIZE_OVERFLOW_WITH_ACCESS 22022 +#define __WARNING_ALLOC_SIZE_OVERFLOW_IN_ALLOC_WRAPPER 22023 +#define __WARNING_ALLOC_SIZE_UNDERFLOW_IN_ALLOC_WRAPPER 22024 +#define __WARNING_RETVAL_IS_USED_IN_ALLOC_SIZE_COMPUTATION 22025 +#define __WARNING_REFARG_IS_USED_IN_ALLOC_SIZE_COMPUTATION 22026 +#define __WARNING_OVERFLOW_OR_UNDERFLOW_IN_ALLOCATOR_SIZE_FUNC_RETVAL 22027 +#define __WARNING_OVERFLOW_OR_UNDERFLOW_IN_ALLOCATOR_SIZE_FUNC_REFARG 22028 +#define __WARNING_OVERFLOW_OR_UNDERFLOW_IN_ALLOC_SIZE 22029 +#define __WARNING_OVERFLOW_OR_UNDERFLOW_IN_ALLOC_WRAPPER 22030 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_OVERFLOW 22051 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_WRONG_OVERFLOW_CHECK 22052 +#define __WARNING_UNTRUSTED_INDEX_WRONG_OVERFLOW_CHECK 22053 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_NEGATIVE 22054 +#define __WARNING_UNTRUSTED_INDEX_NEGATIVE 22055 +#define __WARNING_UNTRUSTED_LOOP_INDEX_WRONG_OVERFLOW_CHECK 22056 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_UNDERFLOW 22057 +#define __WARNING_UNTRUSTED_INDEX_UNDERFLOW 22058 +#define __WARNING_UNTRUSTED_LOOP_INDEX_UNDERFLOW 22059 +#define __WARNING_UNTRUSTED_SIGNED_UNSIGNED_COMPARISON 22060 +#define __WARNING_UNTRUSTED_DIFF_SIZE_SIGNED_UNSIGNED_COMPARISON 22061 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_OVERFLOW_WITH_ACCESS 22062 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_OVERFLOW_IN_ALLOC_WRAPPER 22063 +#define __WARNING_UNTRUSTED_ALLOC_SIZE_UNDERFLOW_IN_ALLOC_WRAPPER 22064 +#define __WARNING_UNTRUSTED_OVERFLOW_OR_UNDERFLOW_IN_ALLOCATOR_SIZE_FUNC_RETVAL 22067 +#define __WARNING_UNTRUSTED_OVERFLOW_OR_UNDERFLOW_IN_ALLOCATOR_SIZE_FUNC_REFARG 22068 +#define __WARNING_UNTRUSTED_OVERFLOW_OR_UNDERFLOW_IN_ALLOC_SIZE 22069 +#define __WARNING_UNTRUSTED_OVERFLOW_OR_UNDERFLOW_IN_ALLOC_WRAPPER 22070 +#define __WARNING_UNTRUSTED_ALLOC_SIZE 22081 +#define __WARNING_UNTRUSTED_INDEX 22082 +#define __WARNING_UNTRUSTED_LOOP_INDEX 22083 +#define __WARNING_UNTRUSTED_OVERFLOWN_EXPR 22084 +#define __WARNING_UNTRUSTED_UNDERFLOWN_EXPR 22085 +#define __WARNING_UNRELATED_SIZE_BUFFER_ACCESS 22101 +#define __WARNING_UNRELATED_LOOP_TERMINATION 22102 +#define __WARNING_UNRELATED_LOOP_TERMINATION_NO_SIZEEXPR 22103 +#define __WARNING_BUFFER_COPY_NO_KNOWN_SIZEEXPR 22104 +#define __WARNING_BUFFER_COPY_NO_PREDICATE 22105 +#define __WARNING_BUFFER_COPY_SOURCE_LEN 22106 +#define __WARNING_BUFFER_COPY_STRUCT_FIELD_ANNOTATION 22107 +#define __WARNING_SIZEEXPR_NOT_REINIT 22108 +#define __WARNING_MOVE_OBJECT_WITH_VTABLE_TO_HEAP 22109 +#define __WARNING_ENCODE_GLOBAL_FUNCTION_POINTER 22110 +#define __WARNING_ENCODE_GLOBAL_DISPATCH_TABLE 22111 +#define __WARNING_ENCODE_MEMBER_FUNCTION_POINTER 22112 +#define __WARNING_ENCODE_MEMBER_DISPATCH_TABLE 22113 +#define __WARNING_MISSING_ENCODE_ASSIGNMENT 22114 +#define __WARNING_MISSING_DECODE_CALL 22115 +#define __WARNING_BUFFER_COPY_FUNCTION_NO_PREDICATE 22116 +#define __WARNING_SENDMESSAGE_WITHOUT_CHECKING_BUFFER_SIZE 22117 +#define __WARNING_MISMATCH_RETURN_TYPE 23601 +#define __WARNING_MISMATCH_INITEXPR_TYPE 23602 +#define __WARNING_MISMATCH_ASSIGNMENT_TYPE 23603 +#define __WARNING_MISMATCH_FORMAL_TYPE_AT_CALL 23604 +#define __WARNING_MISMATCH_ADD_PROPERTY 23605 +#define __WARNING_MISMATCH_REMOVE_PROPERTY 23606 +#define __WARNING_MISMATCH_POST_TYPE 23607 +#define __WARNING_MISMATCH_FORMAL_VALID_AT_CALL 23608 +#define __WARNING_MISMATCH_FORMAL_VALIDSRC_AT_CALL 23609 +#define __WARNING_MISMATCH_REFERENCE_VALID 23610 +#define __WARNING_MISMATCH_REFERENCE_VALIDSRC 23611 +#define __WARNING_MISMATCH_MEMBER_VALID 23612 +#define __WARNING_MISMATCH_MEMBER_VALIDSRC 23613 +#define __WARNING_MISMATCH_IN_UNTRUSTED_DATA_SPECS 23614 +#define __WARNING_POTENTIAL_FILE_READ 23615 +#define __WARNING_CONSUMES_UNTRUSTED_DATA 23616 +#define __WARNING_MISMATCH_PUBLIC_FORMAL_VALID_AT_CALL 23617 +#define __WARNING_FILE_PARSER_CLASS 23620 +#define __WARNING_FILE_PARSER_FUNCTION 23621 +#define __WARNING_MISMATCH_RETURN_TYPE_COMPNAME 23622 +#define __WARNING_MISMATCH_INITEXPR_TYPE_COMPNAME 23623 +#define __WARNING_MISMATCH_ASSIGNMENT_TYPE_COMPNAME 23624 +#define __WARNING_MISMATCH_FORMAL_TYPE_AT_CALL_COMPNAME 23625 +#define __WARNING_MISMATCH_ADD_PROPERTY_COMPNAME 23626 +#define __WARNING_MISMATCH_REMOVE_PROPERTY_COMPNAME 23627 +#define __WARNING_MISMATCH_POST_TYPE_COMPNAME 23628 +#define __WARNING_ALLOCATOR 23629 +#define __WARNING_IPV6_NAME_RESOLUTION_IPV4_SPECIFIC 24001 +#define __WARNING_IPV6_ADDRESS_STRUCTURE_IPV4_SPECIFIC 24002 +#define __WARNING_IPV6_MULTICAST_IPV4_SPECIFIC 24003 +#define __WARNING_IPV6_CONSTANT_IPV4_SPECIFIC 24004 +#define __WARNING_IPV6_OBSOLETE_FUNCTION 24005 +#define __WARNING_IPV6_DEPRECATED_FUNCTION 24006 +#define __WARNING_IPV6_USE_EX_VERSION 24007 +#define __WARNING_IPV6_INVALID_DEVICE 24008 +#define __WARNING_IPV6_DEPRECATED_IOCTL 24009 +#define __WARNING_BIG_FORMAL_PARAM 25000 +#define __WARNING_IMPLICIT_CTOR 25001 +#define __WARNING_UPCAST_CTOR 25002 +#define __WARNING_NONCONST_LOCAL 25003 +#define __WARNING_NONCONST_PARAM 25004 +#define __WARNING_NONCONST_FUNCTION 25005 +#define __WARNING_UNUSED_PARAM 25006 +#define __WARNING_STATIC_FUNCTION 25007 +#define __WARNING_FREE_CONSTRUCTOR_CALL 25008 +#define __WARNING_IDENTITY_ASSIGNMENT 25009 +#define __WARNING_VIRTUAL_CALL_IN_CTOR 25010 +#define __WARNING_CASE_FALLTHRU 25011 +#define __WARNING_SMARTPOINTER_DELETE 25012 +#define __WARNING_RETURN_FROM_GOTO_CONTEXT 25013 +#define __WARNING_MISSING_OVERRIDE 25014 +#define __WARNING_DOESNT_OVERRIDE 25015 +#define __WARNING_OVERRIDE_CONST_MISMATCH 25016 +#define __WARNING_OVERRIDING_NONVIRTUAL 25017 +#define __WARNING_DIFFERENT_CALLING_CONVENTION 25018 +#define __WARNING_VERYBIG_FORMAL_PARAM 25019 +#define __WARNING_SAMENAME_DATAMEMBER 25020 +#define __WARNING_POOR_DATAALIGNMENT 25021 +#define __WARNING_REMOVERETURNCONST_FUNCTIONCAST 25022 +#define __WARNING_BOGUS_BSTRCONST 25023 +#define __WARNING_DANGEROUS_POINTERCAST 25024 +#define __WARNING_UNSAFE_STRING_FUNCTION 25025 +#define __WARNING_OBSOLETE_INI_ACCESSOR 25026 +#define __WARNING_BANNED_FUNCTION 25027 +#define __WARNING_FUNCTION_NEEDS_REVIEW 25028 +#define __WARNING_OBSOLETE_ACL_FUNCTION 25029 +#define __WARNING_BOGUS_EXPRESSION_LIST 25030 +#define __WARNING_HRESULT_NOT_CHECKED 25031 +#define __WARNING_NONCONST_LOCAL_BUFFERPTR 25032 +#define __WARNING_NONCONST_BUFFER_PARAM 25033 +#define __WARNING_SIZEOF_POINTER_IN_DIVISION 25035 +#define __WARNING_STATIC_MASKING_VIRTUAL 25036 +#define __WARNING_TRUE_CONSTANT_EXPR_IN_AND 25037 +#define __WARNING_FALSE_CONSTANT_EXPR_IN_AND 25038 +#define __WARNING_TRUE_CONSTANT_EXPR_IN_OR 25039 +#define __WARNING_FALSE_CONSTANT_EXPR_IN_OR 25040 +#define __WARNING_IF_CONDITION_IS_ALWAYS_TRUE 25041 +#define __WARNING_IF_CONDITION_IS_ALWAYS_FALSE 25042 +#define __WARNING_LOCAL_BSTR_SHOULD_BE_CONST_WCHAR_PTR 25043 +#define __WARNING_BSTR_PARAM_SHOULD_BE_CONST_WCHAR_PTR 25044 +#define __WARNING_ISBADPTR_FUNCTION 25045 +#define __WARNING_SPECIFY_SELECTANY 25046 +#define __WARNING_INCORRECT_SELECTANY 25047 +#define __WARNING_STRINGCONST_ASSIGNED_TO_NONCONST 25048 +#define __WARNING_FREESTANDING_CONSTANT 25049 +#define __WARNING_LOCAL_FUNCDECL 25050 +#define __WARNING_DIFFERENT_RETURN_TYPE_SIZE 25051 +#define __WARNING_DIFFERENT_RETURN_TYPE_KIND 25052 +#define __WARNING_DIFFERENT_PARAM_COUNT 25053 +#define __WARNING_DIFFERENT_PARAM_TYPE_SIZE 25054 +#define __WARNING_DIFFERENT_PARAM_TYPE_KIND 25055 +#define __WARNING_REMOVEPARAMCONST_FUNCTIONCAST 25056 +#define __WARNING_COUNT_REQUIRED_FOR_WRITABLE_BUFFER 25057 +#define __WARNING_CAST_CAN_BE_CONST 25058 +#define __WARNING_SUPERFLUOUS_CAST 25059 +#define __WARNING_SUPERFLUOUS_NOTNOT 25060 +#define __WARNING_BACKWARD_JUMP 25061 +#define __WARNING_BACKWARD_JUMP_IN_MACRO 25062 +#define __WARNING_FILEMAPPING_WITH_GLOBAL_NAME 25063 +#define __WARNING_FUNCTION_CALLED_TWICE_IN_MACRO 25064 +#define __WARNING_UNSIGNED_SHORT_NEGATIVE_INT_COMPARISON 25065 +#define __WARNING_OUTOFRANGE_SHORT_LONG_COMPARISON 25066 +#define __WARNING_UNREACHABLE_CASE_LABEL 25067 +#define __WARNING_USE_WIDE_API 25068 +#define __WARNING_GOTO 25069 +#define __WARNING_UNINITIALIZED_DATAMEMBER 25070 +#define __WARNING_NO_MEMBERINIT 25071 +#define __WARNING_WRONG_MEMBERINIT_ORDER 25073 +#define __WARNING_NOHEAP_MEMBER 25075 +#define __WARNING_NOHEAP_BASECLASS 25076 +#define __WARNING_NEW_ON_NOHEAP 25077 +#define __WARNING_CONDITIONAL_WITH_FALSE_TRUE 25078 +#define __WARNING_CONDITIONAL_WITH_TRUE_FALSE 25079 +#define __WARNING_CONDITIONAL_WITH_SAME_ALTERNATIVES 25080 +#define __WARNING_DANGEROUS_ALL_ACCESS_ACL 25084 +#define __WARNING_URL_NEEDS_TO_BE_REVIEWED 25085 +#define __WARNING_SD_REQUIRED_FOR_NAMED_OBJECT 25086 +#define __WARNING_UNSPECIFIED_ACCESSRIGHTS 25087 +#define __WARNING_DEPRECATED_WIN16_FUNCTION 25089 +#define __WARNING_USE_CPLUSPLUS_BOOL_CONST 25090 +#define __WARNING_RETURNTYPE_CAN_BE_BOOL 25091 +#define __WARNING_LOCAL_CAN_BE_BOOL 25092 +#define __WARNING_FORMAL_CAN_BE_BOOL 25093 +#define __WARNING_BASECLASS_SHOULDHAVE_PROTECTED_OR_VIRTUAL_DTOR 25094 +#define __WARNING_ENUM_TYPEDEF 25096 +#define __WARNING_INTEGRAL_CAST_TO_OBJECT_WITH_VTABLE 25098 +#define __WARNING_USE_PREFIX_OPERATOR 25099 +#define __WARNING_LHS_TEMP_OBJECT 25100 +#define __WARNING_RHS_TEMP_OBJECT 25101 +#define __WARNING_EXC_NOT_CAUGHT_BY_REFERENCE 25102 +#define __WARNING_EXC_NOT_THROWN_BY_VALUE 25103 +#define __WARNING_INCONSISTENT_DECLSPECS 25104 +#define __WARNING_SEALED_BASE_CLASS 25105 +#define __WARNING_SEALED_BASE_METHOD 25106 +#define __WARNING_NOTRUNCCAST_PARAM 25107 +#define __WARNING_UNBOUND_RETURN 25108 +#define __WARNING_MISSING_BINDRETURN 25109 +#define __WARNING_CLARYFY_PRECEDENCE_FOR_QUESTIONOPERATOR 25110 +#define __WARNING_DEPRECATED_FUNCTION 25112 +#define __WARNING_DEPRECATED_LANGUAGE_TYPE_USED 25113 +#define __WARNING_UNMARKED_INTL_DEPRECATED_FUNCTION 25114 +#define __WARNING_DEPRECATED_INTL_FUNCTION_CALL 25115 +#define __WARNING_OLEO_DEPRECATED_FUNCTION 25119 +#define __WARNING_COUNT_REQUIRED_FOR_VOIDPTR_BUFFER 25120 +#define __WARNING_RETURNING_TEMP_OBJECT 25124 +#define __WARNING_STRCPY_LOOP 25125 +#define __WARNING_POSSIBLE_STRCPY_LOOP 25126 +#define __WARNING_BOOLEAN_BITWISE_OPERATOR_MIX 25127 +#define __WARNING_DIFFERENT_COMPARE_ON_64BIT 25128 +#define __WARNING_POSSIBLE_64BIT_TRUNCATION 25129 +#define __WARNING_NONCONST_CLSID 25130 +#define __WARNING_EMPTY_DTOR 25131 +#define __WARNING_BY_VALUE_FORMAL_WITH_DTOR 25132 +#define __WARNING_BY_VALUE_TEMPLATEFORMAL_WITH_DTOR 25133 +#define __WARNING_IMPLICIT_TEMPLATECTOR 25134 +#define __WARNING_LOCAL_ARRAY_SHOULD_BE_STATIC 25135 +#define __WARNING_NONCONST_LOCAL_ARRAY 25136 +#define __WARNING_LOCAL_ARRAY_SHOULD_BE_PTR 25137 +#define __WARNING_STATIC_ARRAY_SHOULD_BE_LOCAL_PTR 25138 +#define __WARNING_EMPTY_NONPUBLIC_DTOR 25139 +#define __WARNING_DANGEROUS_INTTOPTR_CAST 25140 +#define __WARNING_HUNGARIAN_INFERRED 25351 +#define __WARNING_MISSING_ANNOTATION 25352 +#define __WARNING_OVERRIDE_AT_NON_VIRTUAL 25353 +#define __WARNING_OVERRIDE_AT_STATIC 25354 +#define __WARNING_OVERRIDE_AT_BASECLASS 25355 +#define __WARNING_UNSUPPORTED_DEREF_VALUE 25356 +#define __WARNING_PRE_ANNOTATION_AT_RETURNVALUE 25357 +#define __WARNING_MISSING_NOTHROW_AT_DECL 25358 +#define __WARNING_DEPRECATED_OVERRIDE 25359 +#define __WARNING_OBSOLETE_OVERRIDE 25360 +#define __WARNING_ILLEGAL_BOUND_AT_PARAM 25361 +#define __WARNING_ILLEGAL_BOUND_AT_RETURN 25362 +#define __WARNING_ILLEGAL_DEREF_BOUND_AT_PARAM 25363 +#define __WARNING_ILLEGAL_DEREF_BOUND_AT_RETURN 25364 +#define __WARNING_ILLEGAL_RANGE_AT_PARAM 25365 +#define __WARNING_ILLEGAL_RANGE_AT_RETURN 25366 +#define __WARNING_ILLEGAL_DEREF_RANGE_AT_PARAM 25367 +#define __WARNING_ILLEGAL_DEREF_RANGE_AT_RETURN 25368 +#define __WARNING_PARAM_UPPERBOUND_LT_LOWERBOUND 25369 +#define __WARNING_RETURN_UPPERBOUND_LT_LOWERBOUND 25370 +#define __WARNING_NO_ANNOTATIONS_AT_DECLARATION 25371 +#define __WARNING_DECLSPEC_SAL_ANNOTATION 25372 +#define __WARNING_BUFFER_OVERFLOW 26000 +#define __WARNING_BUFFER_UNDERFLOW 26001 +#define __WARNING_READ_UNTRACKED_BUFFER 26002 +#define __WARNING_UNTRACKED_BUFFER 26003 +#define __WARNING_UNINITIALIZED_POINTER 26004 +#define __WARNING_ZEROLENGTHARRAY 26005 +#define __WARNING_INCORRECT_ANNOTATION_STRING 26006 +#define __WARNING_INCORRECT_ANNOTATION 26007 +#define __WARNING_BUFFERACCESS 26009 +#define __WARNING_POTENTIAL_BUFFER_OVERFLOW 26010 +#define __WARNING_POTENTIAL_BUFFER_UNDERFLOW 26011 +#define __WARNING_UNTRACKED_BUFFER_BUT_UNANNOTATABLE 26012 +#define __WARNING_COMPLEX_EXPR 26013 +#define __WARNING_INCORRECT_VALIDATION 26014 +#define __WARNING_POTENTIAL_BUFFER_OVERFLOW_HIGH_PRIORITY 26015 +#define __WARNING_BUFFER_OVERFLOW_NULL_TERMINATED 26016 +#define __WARNING_POTENTIAL_BUFFER_OVERFLOW_MISC 26017 +#define __WARNING_POTENTIAL_BUFFER_OVERFLOW_NULLTERMINATED 26018 +#define __WARNING_INCORRECT_VALIDATION2 26019 +#define __WARNING_INSANE_ANNOTATION 26020 +#define __WARNING_ANNOTATION_MISMATCH_MISSING_FIRST 26021 +#define __WARNING_ANNOTATION_MISMATCH_MISSING_SECOND 26022 +#define __WARNING_ANNOTATION_MISMATCH 26023 +#define __WARNING_ANNOTATION_MISMATCH_INTERNAL 26029 +#define __WARNING_POSTCONDITION_BUFFER_OVERFLOW 26030 +#define __WARNING_POSTCONDITION_BUFFER_UNDERFLOW 26031 +#define __WARNING_PRECONDITION_NULLTERMINATION_VIOLATION 26035 +#define __WARNING_POSTCONDITION_NULLTERMINATION_VIOLATION 26036 +#define __WARNING_POTENTIAL_NULLTERMINATION_VIOLATION 26037 +#define __WARNING_POTENTIAL_POSTCONDITION_BUFFER_OVERFLOW 26040 +#define __WARNING_POTENTIAL_POSTCONDITION_BUFFER_UNDERFLOW 26041 +#define __WARNING_INCORRECT_VALIDATION_POSTCONDITION 26044 +#define __WARNING_HIGH_PRIORITY_OVERFLOW_POSTCONDITION 26045 +#define __WARNING_CHECKER_ASSERTION_FAILURE 26050 +#define __WARNING_IRREDUCIBLE_CFG 26051 +#define __WARNING_POTENTIALLY_UNCONSTRAINED_CALL 26052 +#define __WARNING_POTENTIAL_BUFFER_OVERFLOW_LOOP_DEPENDENT 26053 +#define __WARNING_RANGE_PRECONDITION_VIOLATION 26060 +#define __WARNING_RANGE_POSTCONDITION_VIOLATION 26061 +#define __WARNING_POTENTIAL_RANGE_PRECONDITION_VIOLATION 26070 +#define __WARNING_POTENTIAL_RANGE_POSTCONDITION_VIOLATION 26071 +#define __WARNING_VALIDATED_ACCESS 26080 +#define __WARNING_USING_STRUCT_ANNOTATION 26081 +#define __WARNING_SUSPICIOUS_FUNCTION_POINTER_CAST 26090 +#define __WARNING_RACE_CONDITION 26100 +#define __WARNING_INTERLOCKED_VIOLATION 26101 +#define __WARNING_RACE_CONDITION_VIA_PARAMETER 26102 +#define __WARNING_LOCKORDER_ACQUIRE_VIOLATION 26105 +#define __WARNING_LOCKORDER_RELEASE_VIOLATION 26106 +#define __WARNING_CALLER_FAILING_TO_HOLD 26110 +#define __WARNING_CALLER_FAILING_TO_RELEASE 26111 +#define __WARNING_CALLER_CANNOT_HOLD_ANY_LOCK 26112 +#define __WARNING_FAILING_TO_RELEASE 26115 +#define __WARNING_FAILING_TO_ACQUIRE 26116 +#define __WARNING_RELEASING_UNHELD_LOCK 26117 +#define __WARNING_DOUBLE_LOCK 26118 +#define __WARNING_NEED_NO_COMPETING_THREAD 26130 +#define __WARNING_NEED_REQUIRES_LOCK_HELD 26131 +#define __WARNING_NEED_SIDE_EFFECT 26135 +#define __WARNING_NEED_GUARDED_BY 26136 +#define __WARNING_NEED_INTERLOCKED 26137 +#define __WARNING_NEED_LOCK_ORDER 26138 +#define __WARNING_NEED_LOCK_PROPERTY 26139 +#define __WARNING_CSAL_ANNOTATION_ERROR 26140 +#define __WARNING_IGNORE_LOCK_CHECKING 26149 +#define __WARNING_RACE_CONDITION_MEDIUM_CONFIDENCE 26150 +#define __WARNING_INTERLOCKED_VIOLATION_MEDIUM_CONFIDENCE 26151 +#define __WARNING_RACE_CONDITION_VIA_PARAMETER_MEDIUM_CONFIDENCE 26152 +#define __WARNING_LOCKORDER_ACQUIRE_VIOLATION_MEDIUM_CONFIDENCE 26155 +#define __WARNING_LOCKORDER_RELEASE_VIOLATION_MEDIUM_CONFIDENCE 26156 +#define __WARNING_CALLER_FAILING_TO_HOLD_MEDIUM_CONFIDENCE 26160 +#define __WARNING_CALLER_FAILING_TO_RELEASE_MEDIUM_CONFIDENCE 26161 +#define __WARNING_CALLER_CANNOT_HOLD_ANY_LOCK_MEDIUM_CONFIDENCE 26162 +#define __WARNING_FAILING_TO_RELEASE_MEDIUM_CONFIDENCE 26165 +#define __WARNING_FAILING_TO_ACQUIRE_MEDIUM_CONFIDENCE 26166 +#define __WARNING_RELEASING_UNHELD_LOCK_MEDIUM_CONFIDENCE 26167 +#define __WARNING_DOUBLE_LOCK_MEDIUM_CONFIDENCE 26168 +#define __WARNING_NEED_NO_COMPETING_THREAD_MEDIUM_CONFIDENCE 26180 +#define __WARNING_NEED_REQUIRES_LOCK_HELD_MEDIUM_CONFIDENCE 26181 +#define __WARNING_NEED_SIDE_EFFECT_MEDIUM_CONFIDENCE 26185 +#define __WARNING_NEED_GUARDED_BY_MEDIUM_CONFIDENCE 26186 +#define __WARNING_NEED_INTERLOCKED_MEDIUM_CONFIDENCE 26187 +#define __WARNING_NEED_LOCK_ORDER_MEDIUM_CONFIDENCE 26188 +#define __WARNING_NEED_LOCK_PROPERTY_MEDIUM_CONFIDENCE 26189 +#define __WARNING_DRIVER_FUNCTION_TYPE 28101 +#define __WARNING_RESOURCE_LEAK 28103 +#define __WARNING_RESOURCE_NOT_ACQUIRED 28104 +#define __WARNING_RESOURCE_LEAK_EXCEPTION 28105 +#define __WARNING_ALREADY_HELD 28106 +#define __WARNING_MUST_HOLD 28107 +#define __WARNING_WRONG_KIND 28108 +#define __WARNING_MUST_NOT_HOLD 28109 +#define __WARNING_FLOAT_IN_DRIVER 28110 +#define __WARNING_FP_IRQ_UNMATCHED 28111 +#define __WARNING_INTERLOCKED_ACCESS 28112 +#define __WARNING_INTERLOCKED_SCLASS 28113 +#define __WARNING_IRP_COPY 28114 +#define __WARNING_IRQ_TOO_LOW 28120 +#define __WARNING_IRQ_TOO_HIGH 28121 +#define __WARNING_INFERRED_IRQ_TOO_LOW 28122 +#define __WARNING_INFERRED_IRQ_TOO_HIGH 28123 +#define __WARNING_IRQ_SET_TOO_LOW 28124 +#define __WARNING_PROBE_NO_TRY 28125 +#define __WARNING_OBJ_REFERENCE_MODE 28126 +#define __WARNING_PROTOTYPE_MISMATCH 28127 +#define __WARNING_FUNCTION_ASSIGNMENT 28128 +#define __WARNING_MUST_BE_BITOP 28129 +#define __WARNING_NOT_COPYING_NAME 28131 +#define __WARNING_SIZEOF_POINTER 28132 +#define __WARNING_INITIALIZE_TIMER 28133 +#define __WARNING_POOL_TAG 28134 +#define __WARNING_KE_WAIT_LOCAL 28135 +#define __WARNING_ALWAYS_CONSTANT 28137 +#define __WARNING_NEVER_CONSTANT 28138 +#define __WARNING_STRICT_TYPE_MATCH 28139 +#define __WARNING_CRITICAL_OR_IRQ 28140 +#define __WARNING_IRQ_LOWERED_IMPROPERLY 28141 +#define __WARNING_RELEASE_CONSTANT 28142 +#define __WARNING_PENDING_STATUS_ERROR 28143 +#define __WARNING_CANCEL_IRQL_FROM_IRP 28144 +#define __WARNING_MODIFYING_MDL 28145 +#define __WARNING_STRSAFE_H 28146 +#define __WARNING_UNHELPFUL_TAG 28147 +#define __WARNING_IRQ_SET_TOO_HIGH 28150 +#define __WARNING_BAD_IRQL_VALUE 28151 +#define __WARNING_INIT_NOT_CLEARED 28152 +#define __WARNING_MODEL_BAD_IRQL 28153 +#define __WARNING_CLASS_MISMATCH_NONE 28155 +#define __WARNING_IRQ_INCORRECT 28156 +#define __WARNING_IRQL_NOT_USED 28157 +#define __WARNING_IRQL_NOT_SET 28158 +#define __WARNING_USE_OTHER_FUNCTION 28159 +#define __WARNING_ERROR 28160 +#define __WARNING_FLOAT_UNSAFE_EXIT 28161 +#define __WARNING_FLOAT_SAFE_EXIT 28162 +#define __WARNING_MUST_NOT_TRY 28163 +#define __WARNING_PVOID 28164 +#define __WARNING_CLASS_MISMATCH 28165 +#define __WARNING_NOT_SAME_IRQL 28166 +#define __WARNING_UNEXPECTED_IRQL_CHANGE 28167 +#define __WARNING_DISPATCH_MISMATCH 28168 +#define __WARNING_DISPATCH_MISSING 28169 +#define __WARNING_NO_PAGED_CODE 28170 +#define __WARNING_MULTIPLE_PAGED_CODE 28171 +#define __WARNING_NO_PAGING_SEGMENT 28172 +#define __WARNING_GET_ADAPTER_NOT_RETRIED 28173 +#define __WARNING_INACCESSIBLE_MEMBER 28175 +#define __WARNING_READ_ONLY_MEMBER 28176 +#define __WARNING_MULTIPLE_FUNCTION_CLASS 28177 +#define __WARNING_DEREF_INTERMEDIATE_NULL_PTR 28181 +#define __WARNING_DEREF_RENAMED_NULL_PTR 28182 +#define __WARNING_INVALID_PARAM_VALUE_3 28183 +#define __WARNING_DEREF_PARAMETER_NULL_PTR 28184 +#define __WARNING_INVALID_PARAM_VALUE_4 28185 +#define __WARNING_MUST_HOLD_CLASS 28191 +#define __WARNING_MUST_NOT_HOLD_CLASS 28192 +#define __WARNING_MUST_USE 28193 +#define __WARNING_MEMORY_NOT_ALIASED 28194 +#define __WARNING_MEMORY_NOT_ACQUIRED 28195 +#define __WARNING_RETURNING_BAD_RESULT 28196 +#define __WARNING_ALIASED_MEMORY_LEAK 28197 +#define __WARNING_ALIASED_MEMORY_LEAK_EXCEPTION 28198 +#define __WARNING_MAYBE_UNINIT_VAR 28199 +#define __WARNING_INTERNAL_DEBUG 28200 +#define __WARNING_XML_ANNOTATION 28201 +#define __WARNING_INVALID_ANNOTATION_SYMBOL 28209 +#define __WARNING_POST_EXPECTED 28210 +#define __WARNING_CONTEXT_EXPECTED 28211 +#define __WARNING_POINTER_EXPECTED 28212 +#define __WARNING_BAD_USEHEADER 28213 +#define __WARNING_BAD_PARAM_NAME 28214 +#define __WARNING_DUPLICATE_TYPEFIX 28215 +#define __WARNING_MUSTCHECK_NOT_POST 28216 +#define __WARNING_UNMATCHED_ANNO_COUNT 28217 +#define __WARNING_UNMATCHED_ANNO_TREE 28218 +#define __WARNING_ENUM_EXPECTED 28219 +#define __WARNING_INTEGER_EXPECTED 28220 +#define __WARNING_STRING_EXPECTED 28221 +#define __WARNING_YNM_EXPECTED 28222 +#define __WARNING_AUTOQUOTE_EXPECTED 28223 +#define __WARNING_PARAMS_EXPECTED 28224 +#define __WARNING_BAD_NO_PARAMS 28225 +#define __WARNING_NOT_BOTH 28226 +#define __WARNING_NOT_BOTH_2 28227 +#define __WARNING_UNRECOGNIZED_TYPE 28228 +#define __WARNING_NO_PARAMS_EXPECTED 28229 +#define __WARNING_NO_SUCH_MEMBER 28230 +#define __WARNING_ILLEGAL_INDEX 28231 +#define __WARNING_DANGLING_QUALIFIER 28232 +#define __WARNING_BLOCK_QUALIFIER 28233 +#define __WARNING_AT_NOT_ROOT 28234 +#define __WARNING_PRIMOP_AS_ANNOTATION 28235 +#define __WARNING_ANNOTATION_AS_PRIMOP 28236 +#define __WARNING_OBSOLETE 28237 +#define __WARNING_TOOMANY_OLD 28238 +#define __WARNING_MIXED_OLD_NEW 28239 +#define __WARNING_NO_PARAM1 28240 +#define __WARNING_EXTERNAL_UNKNOWN_ANNO 28241 +#define __WARNING_BAD_MULTIPLIER 28242 +#define __WARNING_BAD_DEREF 28243 +#define __WARNING_BAD_EXTERNAL 28244 +#define __WARNING_THIS_NONMEMBER 28245 +#define __WARNING_ANNOTATION_TYPE_MISMATCH 28246 +#define __WARNING_XML_DUPLICATE 28247 +#define __WARNING_XML_MODEL_FILE_DUPLICATE 28248 +#define __WARNING_UNMATCHED_DECL 28250 +#define __WARNING_UNMATCHED_DEFN 28251 +#define __WARNING_UNMATCHED_DECL_ANNO 28252 +#define __WARNING_UNMATCHED_DEFN_ANNO 28253 +#define __WARNING_DYNAMIC_CAST 28254 +#define __WARNING_MODEL_SYNTAX_ERROR 28260 +#define __WARNING_MODEL_SYNTAX_ERROR1 28261 +#define __WARNING_MODEL_SYNTAX_ERROR2 28262 +#define __WARNING_MUST_BE_CONSTANT 28264 +#define __WARNING_MODEL_PLACEMENT_ERROR 28266 +#define __WARNING_MODEL_PLACEMENT_ERROR1 28267 +#define __WARNING_CLASS_MODEL_MISMATCH 28268 +#define __WARNING_MISMATCHED_LIST 28272 +#define __WARNING_MISMATCHED_PREDICATE 28273 +#define __WARNING_MISMATCHED_PREDICATE1 28274 +#define __WARNING_NULL_MACRO_VALUE 28275 +#define __WARNING_NO_MACROS 28276 +#define __WARNING_NO_PROTOTYPE 28278 +#define __WARNING_UNMATCHED_BEGIN 28279 +#define __WARNING_UNMATCHED_END 28280 +#define __WARNING_BAD_INVALID 28281 +#define __WARNING_FORMAT_NOT_PRE 28282 +#define __WARNING_UNSUPPORTED_SIZESPEC 28283 +#define __WARNING_SPEC_INVALID_PREDICATE 28284 +#define __WARNING_SPEC_INVALID_SYNTAX2 28285 +#define __WARNING_SPEC_INVALID_SYNTAX 28286 +#define __WARNING_INVALID_AT2 28287 +#define __WARNING_INVALID_AT 28288 +#define __WARNING_LIMITSPEC_EXPECTED 28289 +#define __WARNING_SPEC_TOO_MANY 28290 +#define __WARNING_LOD_0_NULL 28291 +#define __WARNING_MIN_SIZE_MULTIPLE 28292 +#define __WARNING_MIN_SIZE_UNKNOWN 28293 +#define __WARNING_EXTERNAL_UNKNOWN_PARAM 28294 +#define __WARNING_EXTERNAL_UNKNOWN_TYPE 28295 +#define __WARNING_MODEL_PARSE_CRASH 28296 +#define __WARNING_MODEL_PARSE_1 28297 +#define __WARNING_MODEL_TOO_MANY 28299 +#define __WARNING_AVOID_BLOCKING_HWNDBROADCAST 28601 +#define __WARNING_AVOID_TIMEOUT_HWNDBROADCAST 28602 +#define __WARNING_AVOID_SENDMESSAGE_TIMEOUT_MISUSE 28604 +#define __WARNING_ALLOCA_RESETSTKOFLW 28615 +#define __WARNING_INTERLOCKEDDECREMENT_MISUSE1 28616 +#define __WARNING_AVOID_BEGIN_THREAD_RETURN_VALUE 28617 +#define __WARNING_UNSIGNED_CAST_OF_GETMESSAGEPOS 28623 +#define __WARNING_UNRELEASED_REF 28624 +#define __WARNING_COMPILER_OPTIMIZATES_OUT_CLEARING_SENSITIVEDATA 28625 +#define __WARNING_LOCAL_FREE_NONALLOCATED 28636 +#define __WARNING_UNSAFE_CALL_IN_GLOBAL_INIT 28637 +#define __WARNING_DLOAD_FUNCDEFN 28638 +#define __WARNING_CLOSEHANDLE_STRING 28639 +#define __WARNING_DLOAD_FUNCSTATIC 28640 +#define __WARNING_BAD_POSTMESSAGE_FLAG 28642 +#define __WARNING_DPA_INSERTPTR_RETURN 28644 +#define __WARNING_OBSOLETE_MESSAGE_SYMBOL_QUESTION 28645 +#define __WARNING_DO_NOT_USE_PULSEEVENT 28648 +#define __WARNING_REDUNDANT_CHECK_FOR_ARRAY_IS_NOT_NULL 28649 +#define __WARNING_IMPROPER_NOT_OPERATOR_ON_ZERO 28650 +#define __WARNING_NONSTATIC_FUNCPTR_IN_STATIC_INIT 28651 +#define __WARNING_BITOP_CALL_IN_STATIC_INIT 28652 +#define __WARNING_CAST_NTSTATUS_TO_BOOL 28714 +#define __WARNING_CAST_BOOL_TO_NTSTATUS 28715 +#define __WARNING_COMPILER_INSERTED_CAST_BOOL_TO_NTSTATUS 28716 +#define __WARNING_INVALID_VARIANT_TYPE 28717 +#define __WARNING_UNANNOTATED_BUFFER 28718 +#define __WARNING_BANNED_API_USAGE 28719 +#define __WARNING_ISBADXXXPTR_API_USAGE 28720 +#define __WARNING_DEPRECATED_PERF_COUNT 28721 +#define __WARNING_UNANNOTATED_BUFFER_IN_DECL 28722 +#define __WARNING_UNANNOTATED_BUFFER_IN_DEFN_WO_DECL 28723 +#define __WARNING_SETUNHANDLEDEXCEPTIONFILTER_USE 28725 +#define __WARNING_BANNED_API_USAGEL2 28726 +#define __WARNING_BANNED_API_USAGEL3 28727 +#define __WARNING_BANNED_API_USAGEL4 28728 +#define __WARNING_INCORRECT_NULLTERM_ASSIGN 28730 +#define __WARNING_RETAIL_DEBUGBREAK 28731 +#define __WARNING_THROW_DECL_MISMATCH 28732 +#define __WARNING_THROW_DECL_MISSING 28733 +#define __WARNING_CYCLOMATIC_COMPLEXITY 28734 +#define __WARNING_BANNED_CRIMSON_API_USAGE 28735 +#define __WARNING_BANNED_API_ARGUMENT_USAGE 28736 +#define __WARNING_UNANNOTATED_BUFFER_UCHAR 28740 +#define __WARNING_UNANNOTATED_BUFFER_PROTOTYPE 28741 +#define __WARNING_UNANNOTATED_BUFFER_PROTOTYPE_UCHAR 28742 +#define __WARNING_OUT_NOTNULL_ON_FAILURE 28901 +#define __WARNING_QI_SUCCESS_NO_ADDREF 28903 +#define __WARNING_BUFFER_SIZE_EQUALS_COUNT 28910 +#define __WARNING_REDUNDANT_TEST_IN_MACRO 28920 +#define __WARNING_REDUNDANT_POINTER_TEST_CLEANUP 28921 +#define __WARNING_REDUNDANT_POINTER_TEST 28922 +#define __WARNING_REDUNDANT_SCALAR_TEST 28923 +#define __WARNING_REDUNDANT_POINTER_TEST_FAR_EVIDENCE 28924 +#define __WARNING_POTENTIALLY_REDUNDANT_GLOBAL_POINTER_TEST 28925 +#define __WARNING_UNUSED_POINTER_ASSIGNMENT 28930 +#define __WARNING_UNUSED_SCALAR_ASSIGNMENT 28931 +#define __WARNING_UNUSED_FORMALPARAM_ASSIGNMENT 28933 +#define __WARNING_UNUSED_PTR2PTR_FORMALPARAM_ASSIGNMENT 28934 +#define __WARNING_UNUSED_OUT_FORMALPARAM_ASSIGNMENT 28935 +#define __WARNING_UNREACHABLE_CODE 28940 +#define __WARNING_UNANNONATED_EXP_IN_VOLATILE_CONTEXT 28952 +#define __WARNING_UNEXPECED_LEVEL_IN_VOLATILE_CONTEXT 28953 +#define __WARNING_MULTIPLE_VOLATILE_ACCESS 28954 +#define __WARNING_MULTIPLE_VOLATILE_ACCESS_FROM_ANNOT 28955 +#define __WARNING_COMPILER_GENERATED_VOLATILE_CAST 28956 +#define __WARNING_LOCAL_VARIABLE_IN_VOLATILE_CONTEXT 28957 +#define __WARNING_MULTIPLE_VOLATILE_ACCESS_WITH_CALLS 28958 +#define __WARNING_MULTIPLE_VOLATILE_ACCESS_FROM_ANNOT_WITH_CALLS 28959 +#define __WARNING_DEPRECATED_MSXML4 30001 +#define __WARNING_ACCESSIBILITY_COLORAPI 30022 +#define __WARNING_ACCESSIBILITY_CLIPCURSOR 30023 +#define __WARNING_TEMP_FILE 33700 +#define __WARNING_ZERO_DIVISION 33701 +#define __WARNING_ZERO_DIVISION_WITH_FIELD 33702 +#define __WARNING_PARAMETER_BUFFER_TO_STRUCT_CAST 33703 +#define __WARNING_MEMBER_BUFFER_TO_STRUCT_CAST 33704 +#define __WARNING_SAFE_AFTER_UNSAFE_ARITHMETIC 33705 +#define __WARNING_UNSAFE_AFTER_SAFE_ARITHMETIC 33706 +#define __WARNING_PARAMETER_BUFFER_IN_MEMCPY 33707 +#define __WARNING_MEMBER_BUFFER_IN_MEMCPY 33708 +#define __WARNING_PARAMETER_BUFFER_IN_WRITE_CALL 33709 +#define __WARNING_PARAMETER_BUFFER_IN_READ_CALL 33710 +#define __WARNING_MEMBER_BUFFER_IN_WRITE_CALL 33711 +#define __WARNING_MEMBER_BUFFER_IN_READ_CALL 33712 +#define __WARNING_PARAMETER_BUFFER_UPDATED_INSIDE_LOOP 33713 +#define __WARNING_PARAMETER_BUFFER_UPDATED_OUTSIDE_LOOP 33714 +#define __WARNING_MEMBER_BUFFER_UPDATED_INSIDE_LOOP 33715 +#define __WARNING_MEMBER_BUFFER_UPDATED_OUTSIDE_LOOP 33716 +#define __WARNING_NDIS_DRIVER_PARAMETER_BUFFER 33801 +#define __WARNING_USE_FILE_DEVICE_SECURE_OPEN 33802 +#define __WARNING_USE_SUFFIXED_CALL 33803 +#define __WARNING_USE_RPC_IF_ALLOW_SECURE_ONLY 33804 +#define __WARNING_X_INVALID_ATTRIBUTE_PROPERTY 36500 +#define __WARNING_X_CONFLICTING_ATTRIBUTE_PROPERTY_VALUES 36501 +#define __WARNING_X_REFERENCES_CANT_BE_NULL 36503 +#define __WARNING_X_NULL_ON_NON_POINTER 36504 +#define __WARNING_X_MUSTCHECK_ON_VOID 36505 +#define __WARNING_X_BUFFER_SIZE_ON_NON_POINTER_OR_ARRAY 36506 +#define __WARNING_X_NULL_MISMATCH_AT_DEREF_0 36507 +#define __WARNING_X_WRITE_ACCESS_ON_CONST 36508 +#define __WARNING_X_RETURN_USED_ON_PRECONDITION 36509 +#define __WARNING_X_NULLTERMINATED_ON_NON_POINTER 36510 +#define __WARNING_X_MUSTCHECK_MAYBE 36511 +#define __WARNING_X_NULL_CONFLICTS_WITH_NOT_VALID 36512 +#define __WARNING_X_ELEMENT_SIZE_WITHOUT_BUFFER_SIZE 36513 +#define __WARNING_X_BUFFER_SIZE_EXCEEDS_ARRAY_SIZE 36514 +#define __WARNING_X_BUFFER_SIZE_ON_NON_POINTER 36515 +#define __WARNING_X_NO_PROPERTIES_ON_ATTRIBUTE 36516 +#define __WARNING_X_VALID_SIZE_ON_NON_READABLE_BUFFER 36517 +#define __WARNING_X_WRITABLE_SIZE_ON_NON_WRITABLE_BUFFER 36518 +#define __WARNING_X_NEEDSRELEASE_MAYBE 36519 +#define __WARNING_X_INVALID_SIZE_STRING_DEREF 36521 +#define __WARNING_X_INVALID_SIZE_STRING_TYPE 36522 +#define __WARNING_X_INVALID_SIZE_STRING_PARAM 36523 +#define __WARNING_X_INVALID_SIZE_STRING_UNREACHABLE_LOCATION 36525 +#define __WARNING_X_INVALID_SIZE_STRING_BUFFER_TYPE 36526 +#define __WARNING_X_NEEDSRELEASE_ON_VOID 36527 +#define __WARNING_X_UNRECOGNIZED_FORMAT_STRING_STYLE 36530 +#define __WARNING_X_BUFFERSIZE_INFEASIBLE 36535 +#define __WARNING_X_OVERRIDING_OLD_MODEL 36540 +#define __WARNING_X_INVALID_SIZE_STRING_EXPRESSION 36551 +#define __WARNING_X_INVALID_DEREF_EXPRESSION 36552 +#define __WARNING_LOCALE_SENSITIVE_STRCMP 38001 +#define __WARNING_LOCALE_SENSITIVE_COMPARESTRING 38002 +#define __WARNING_SYSTEM_LOCALE_MISUSE 38003 +#define __WARNING_DEPRECATED_LIBRARY 38004 +#define __WARNING_CALLING_SETTHREADLOCALE 38010 +#define __WARNING_OVERRIDING_LOCALE_SETTINGS_WITH_SETLOCALEINFO 38011 +#define __WARNING_ANSI_APICALL 38020 +#define __WARNING_W2A_BEST_FIT 38021 +#define __WARNING_CLIPBOARD_ANSI 38022 +#define __WARNING_ANSI_DATAFILE 38023 +#define __WARNING_HARD_CODED_STRING_TO_UI_FN 38030 +#define __WARNING_HARDCODED_FONT_INFO 38031 +#define __WARNING_CONSOLE_OUTPUT_ISSUE 38032 +#define __WARNING_FORMAT_MESSAGE_LANG 38033 +#define __WARNING_CONCATENATED_RESOURCE_STRING 38034 +#define __WARNING_LOCALIZABLE_STRING_FORMAT_ISSUE 38035 +#define __WARNING_MESSAGEBOX_RTL 38036 +#define __WARNING_BITMAP_OR_ANSI_FONT 38037 +#define __WARNING_COMMANDLINK_SETNOTE 38038 +#define __WARNING_SCREEN_COORDINATES_CONVERSION 38039 +#define __WARNING_GETDATEFORMAT_WRAPPER 38041 +#define __WARNING_GETDATEFORMAT_AUTOLAYOUT 38042 +#define __WARNING_LAYOUT_BITMAPORIENTATIONPRESERVED 38043 +#define __WARNING_LOC_METADATA_GENERATED 38044 +#define __WARNING_NON_LOCFRIENDLY_STRING_FORMATTING 38045 + Propchange: trunk/reactos/include/psdk/suppress.h ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/reactos/include/reactos/probe.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/probe.h?re… ============================================================================== --- trunk/reactos/include/reactos/probe.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/probe.h [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -1,5 +1,7 @@ #ifndef INCLUDE_REACTOS_CAPTURE_H #define INCLUDE_REACTOS_CAPTURE_H + +#include <suppress.h> #if ! defined(_NTOSKRNL_) && ! defined(_WIN32K_) #error Header intended for use by NTOSKRNL/WIN32K only! @@ -109,7 +111,7 @@ } /* Probe the array */ - _PRAGMA_WARNING_SUPPRESS(28125) /* Must be inside __try / __except block */ + _PRAGMA_WARNING_SUPPRESS(__WARNING_PROBE_NO_TRY) /* Must be inside __try / __except block */ ProbeForRead(ArrayPtr, ArraySize, Alignment); } @@ -130,7 +132,7 @@ } /* Probe the array */ - _PRAGMA_WARNING_SUPPRESS(28125) /* Must be inside __try / __except block */ + _PRAGMA_WARNING_SUPPRESS(__WARNING_PROBE_NO_TRY) /* Must be inside __try / __except block */ ProbeForWrite(ArrayPtr, ArraySize, Alignment); } #endif /* _WIN32K_ */ Modified: trunk/reactos/win32ss/gdi/ntgdi/path.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/path.c?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -9,6 +9,7 @@ */ #include <win32k.h> +#include <suppress.h> #define NDEBUG #include <debug.h> @@ -1166,11 +1167,11 @@ if((pPath->pFlags[i] & ~PT_CLOSEFIGURE) == PT_MOVETO) { iStroke++; - _PRAGMA_WARNING_SUPPRESS(6386) + _PRAGMA_WARNING_SUPPRESS(__WARNING_WRITE_OVERRUN) pNumPointsInStroke[iStroke]=0; } - _PRAGMA_WARNING_SUPPRESS(6385) + _PRAGMA_WARNING_SUPPRESS(__WARNING_READ_OVERRUN) pNumPointsInStroke[iStroke]++; } Modified: trunk/reactos/win32ss/gdi/ntgdi/region.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/region.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] Sun Dec 16 11:32:56 2012 @@ -114,6 +114,7 @@ */ #include <win32k.h> +#include <suppress.h> #define NDEBUG #include <debug.h> @@ -1841,7 +1842,7 @@ return FALSE; } - _PRAGMA_WARNING_SUPPRESS(28199) // rc is initialized + _PRAGMA_WARNING_SUPPRESS(__WARNING_MAYBE_UNINIT_VAR) // rc is initialized COPY_RECTS(rgn->Buffer, rc, rgn->rdh.nCount); } }

1 0

[tkreuzer] 57921: [RTL] Formatting, no code change
by tkreuzer＠svn.reactos.org 15 Dec '12

15 Dec '12

Author: tkreuzer Date: Sat Dec 15 13:58:24 2012 New Revision: 57921 URL: http://svn.reactos.org/svn/reactos?rev=57921&view=rev Log: [RTL] Formatting, no code change Modified: trunk/reactos/lib/rtl/handle.c Modified: trunk/reactos/lib/rtl/handle.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/handle.c?rev=57921… ============================================================================== --- trunk/reactos/lib/rtl/handle.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/handle.c [iso-8859-1] Sat Dec 15 13:58:24 2012 @@ -2,7 +2,7 @@ * PROJECT: ReactOS system libraries * PURPOSE: Handle table * FILE: lib/rtl/handle.c - * PROGRAMER: Eric Kohl + * PROGRAMMER: Eric Kohl */ /* INCLUDES *****************************************************************/ @@ -14,182 +14,196 @@ /* GLOBALS ******************************************************************/ -VOID NTAPI -RtlInitializeHandleTable(ULONG TableSize, - ULONG HandleSize, - PRTL_HANDLE_TABLE HandleTable) -{ - /* initialize handle table */ - memset(HandleTable, - 0, - sizeof(RTL_HANDLE_TABLE)); - HandleTable->MaximumNumberOfHandles = TableSize; - HandleTable->SizeOfHandleTableEntry = HandleSize; -} - - -/* - * @implemented - */ -VOID NTAPI -RtlDestroyHandleTable(PRTL_HANDLE_TABLE HandleTable) -{ - PVOID ArrayPointer; - SIZE_T ArraySize = 0; - - /* free handle array */ - if (HandleTable->CommittedHandles) - { +VOID +NTAPI +RtlInitializeHandleTable( + ULONG TableSize, + ULONG HandleSize, + PRTL_HANDLE_TABLE HandleTable) +{ + /* Initialize handle table */ + memset(HandleTable, 0, sizeof(RTL_HANDLE_TABLE)); + HandleTable->MaximumNumberOfHandles = TableSize; + HandleTable->SizeOfHandleTableEntry = HandleSize; +} + + +/* + * @implemented + */ +VOID +NTAPI +RtlDestroyHandleTable( + PRTL_HANDLE_TABLE HandleTable) +{ + PVOID ArrayPointer; + SIZE_T ArraySize = 0; + + /* free handle array */ + if (HandleTable->CommittedHandles) + { ArrayPointer = (PVOID)HandleTable->CommittedHandles; NtFreeVirtualMemory(NtCurrentProcess(), - &ArrayPointer, - &ArraySize, - MEM_RELEASE); - } -} - - -/* - * @implemented - */ -PRTL_HANDLE_TABLE_ENTRY NTAPI -RtlAllocateHandle(PRTL_HANDLE_TABLE HandleTable, - PULONG Index) -{ - PRTL_HANDLE_TABLE_ENTRY *pp_new, *pph, ph; - NTSTATUS Status; - PRTL_HANDLE_TABLE_ENTRY retval; - PVOID ArrayPointer; - SIZE_T ArraySize; - - pp_new = &HandleTable->FreeHandles; - - if (HandleTable->FreeHandles == NULL) - { - /* no free handle available */ - if (HandleTable->UnCommittedHandles == NULL) - { - /* allocate handle array */ - ArraySize = HandleTable->SizeOfHandleTableEntry * HandleTable->MaximumNumberOfHandles; - ArrayPointer = NULL; - - /* FIXME - only reserve handles here! */ - Status = NtAllocateVirtualMemory(NtCurrentProcess(), - (PVOID*)&ArrayPointer, - 0, - &ArraySize, - MEM_RESERVE | MEM_COMMIT, - PAGE_READWRITE); - if (!NT_SUCCESS(Status)) - return NULL; - - /* update handle array pointers */ - HandleTable->FreeHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - HandleTable->MaxReservedHandles = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ArrayPointer + ArraySize); - HandleTable->CommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - HandleTable->UnCommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - } + &ArrayPointer, + &ArraySize, + MEM_RELEASE); + } +} + + +/* + * @implemented + */ +PRTL_HANDLE_TABLE_ENTRY +NTAPI +RtlAllocateHandle( + PRTL_HANDLE_TABLE HandleTable, + PULONG Index) +{ + PRTL_HANDLE_TABLE_ENTRY *pp_new, *pph, ph; + NTSTATUS Status; + PRTL_HANDLE_TABLE_ENTRY retval; + PVOID ArrayPointer; + SIZE_T ArraySize; + + pp_new = &HandleTable->FreeHandles; + + if (HandleTable->FreeHandles == NULL) + { + /* no free handle available */ + if (HandleTable->UnCommittedHandles == NULL) + { + /* allocate handle array */ + ArraySize = HandleTable->SizeOfHandleTableEntry * HandleTable->MaximumNumberOfHandles; + ArrayPointer = NULL; + + /* FIXME - only reserve handles here! */ + Status = NtAllocateVirtualMemory(NtCurrentProcess(), + (PVOID*)&ArrayPointer, + 0, + &ArraySize, + MEM_RESERVE | MEM_COMMIT, + PAGE_READWRITE); + if (!NT_SUCCESS(Status)) + return NULL; + + /* update handle array pointers */ + HandleTable->FreeHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + HandleTable->MaxReservedHandles = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ArrayPointer + ArraySize); + HandleTable->CommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + HandleTable->UnCommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + } /* FIXME - should check if handles need to be committed */ - /* build free list in handle array */ - ph = HandleTable->FreeHandles; - pph = pp_new; - while (ph < HandleTable->MaxReservedHandles) - { - *pph = ph; - pph = &ph->NextFree; - ph = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ph + HandleTable->SizeOfHandleTableEntry); - } - *pph = 0; - } - - /* remove handle from free list */ - retval = *pp_new; - *pp_new = retval->NextFree; - retval->NextFree = NULL; - - if (Index) - *Index = ((ULONG)((ULONG_PTR)retval - (ULONG_PTR)HandleTable->CommittedHandles) / - HandleTable->SizeOfHandleTableEntry); - - return retval; -} - - -/* - * @implemented - */ -BOOLEAN NTAPI -RtlFreeHandle(PRTL_HANDLE_TABLE HandleTable, - PRTL_HANDLE_TABLE_ENTRY Handle) + /* build free list in handle array */ + ph = HandleTable->FreeHandles; + pph = pp_new; + while (ph < HandleTable->MaxReservedHandles) + { + *pph = ph; + pph = &ph->NextFree; + ph = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ph + HandleTable->SizeOfHandleTableEntry); + } + *pph = 0; + } + + /* remove handle from free list */ + retval = *pp_new; + *pp_new = retval->NextFree; + retval->NextFree = NULL; + + if (Index) + { + *Index = ((ULONG)((ULONG_PTR)retval - (ULONG_PTR)HandleTable->CommittedHandles) / + HandleTable->SizeOfHandleTableEntry); + } + + return retval; +} + + +/* + * @implemented + */ +BOOLEAN +NTAPI +RtlFreeHandle( + PRTL_HANDLE_TABLE HandleTable, + PRTL_HANDLE_TABLE_ENTRY Handle) { #if DBG - /* check if handle is valid */ - if (!RtlIsValidHandle(HandleTable, Handle)) - { - DPRINT1("Invalid Handle! HandleTable=0x%p, Handle=0x%p, Handle->Flags=0x%x\n", - HandleTable, Handle, Handle ? Handle->Flags : 0); - return FALSE; - } + /* check if handle is valid */ + if (!RtlIsValidHandle(HandleTable, Handle)) + { + DPRINT1("Invalid Handle! HandleTable=0x%p, Handle=0x%p, Handle->Flags=0x%x\n", + HandleTable, Handle, Handle ? Handle->Flags : 0); + return FALSE; + } #endif - /* clear handle */ - memset(Handle, 0, HandleTable->SizeOfHandleTableEntry); - - /* add handle to free list */ - Handle->NextFree = HandleTable->FreeHandles; - HandleTable->FreeHandles = Handle; - - return TRUE; -} - - -/* - * @implemented - */ -BOOLEAN NTAPI -RtlIsValidHandle(PRTL_HANDLE_TABLE HandleTable, - PRTL_HANDLE_TABLE_ENTRY Handle) -{ - if ((HandleTable != NULL) - && (Handle >= HandleTable->CommittedHandles) - && (Handle < HandleTable->MaxReservedHandles) - && (Handle->Flags & RTL_HANDLE_VALID)) - return TRUE; - return FALSE; -} - - -/* - * @implemented - */ -BOOLEAN NTAPI -RtlIsValidIndexHandle(IN PRTL_HANDLE_TABLE HandleTable, - IN ULONG Index, - OUT PRTL_HANDLE_TABLE_ENTRY *Handle) -{ - PRTL_HANDLE_TABLE_ENTRY InternalHandle; - - DPRINT("RtlIsValidIndexHandle(HandleTable %p Index 0x%lx Handle %p)\n", HandleTable, Index, Handle); - - if (HandleTable == NULL) - return FALSE; - - DPRINT("Handles %p HandleSize 0x%lx\n", - HandleTable->CommittedHandles, HandleTable->SizeOfHandleTableEntry); - - InternalHandle = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)HandleTable->CommittedHandles + - (HandleTable->SizeOfHandleTableEntry * Index)); - if (!RtlIsValidHandle(HandleTable, InternalHandle)) - return FALSE; - - DPRINT("InternalHandle %p\n", InternalHandle); - - if (Handle != NULL) - *Handle = InternalHandle; - - return TRUE; + /* clear handle */ + memset(Handle, 0, HandleTable->SizeOfHandleTableEntry); + + /* add handle to free list */ + Handle->NextFree = HandleTable->FreeHandles; + HandleTable->FreeHandles = Handle; + + return TRUE; +} + + +/* + * @implemented + */ +BOOLEAN +NTAPI +RtlIsValidHandle( + PRTL_HANDLE_TABLE HandleTable, + PRTL_HANDLE_TABLE_ENTRY Handle) +{ + if ((HandleTable != NULL) + && (Handle >= HandleTable->CommittedHandles) + && (Handle < HandleTable->MaxReservedHandles) + && (Handle->Flags & RTL_HANDLE_VALID)) + { + return TRUE; + } + return FALSE; +} + + +/* + * @implemented + */ +BOOLEAN +NTAPI +RtlIsValidIndexHandle( + IN PRTL_HANDLE_TABLE HandleTable, + IN ULONG Index, + OUT PRTL_HANDLE_TABLE_ENTRY *Handle) +{ + PRTL_HANDLE_TABLE_ENTRY InternalHandle; + + DPRINT("RtlIsValidIndexHandle(HandleTable %p Index 0x%lx Handle %p)\n", HandleTable, Index, Handle); + + if (HandleTable == NULL) + return FALSE; + + DPRINT("Handles %p HandleSize 0x%lx\n", + HandleTable->CommittedHandles, HandleTable->SizeOfHandleTableEntry); + + InternalHandle = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)HandleTable->CommittedHandles + + (HandleTable->SizeOfHandleTableEntry * Index)); + if (!RtlIsValidHandle(HandleTable, InternalHandle)) + return FALSE; + + DPRINT("InternalHandle %p\n", InternalHandle); + + if (Handle != NULL) + *Handle = InternalHandle; + + return TRUE; } /* EOF */

1 0

[ekohl] 57920: [LSASRV] - Implement LsarRemovePrivilegesFromAccount and LsarOpenPolicy2. - Remove unused Vista+ stubs.
by ekohl＠svn.reactos.org 15 Dec '12

15 Dec '12

Author: ekohl Date: Sat Dec 15 11:31:04 2012 New Revision: 57920 URL: http://svn.reactos.org/svn/reactos?rev=57920&view=rev Log: [LSASRV] - Implement LsarRemovePrivilegesFromAccount and LsarOpenPolicy2. - Remove unused Vista+ stubs. Modified: trunk/reactos/dll/win32/lsasrv/database.c trunk/reactos/dll/win32/lsasrv/lsarpc.c trunk/reactos/dll/win32/lsasrv/lsasrv.h Modified: trunk/reactos/dll/win32/lsasrv/database.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/lsasrv/database.… ============================================================================== --- trunk/reactos/dll/win32/lsasrv/database.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/lsasrv/database.c [iso-8859-1] Sat Dec 15 11:31:04 2012 @@ -1074,5 +1074,14 @@ return Status; } + +NTSTATUS +LsapDeleteObjectAttribute(PLSA_DB_OBJECT DbObject, + LPWSTR AttributeName) +{ + return LsapRegDeleteSubKey(DbObject->KeyHandle, + AttributeName); +} + /* EOF */ Modified: trunk/reactos/dll/win32/lsasrv/lsarpc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/lsasrv/lsarpc.c?… ============================================================================== --- trunk/reactos/dll/win32/lsasrv/lsarpc.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/lsasrv/lsarpc.c [iso-8859-1] Sat Dec 15 11:31:04 2012 @@ -1377,8 +1377,153 @@ BOOL AllPrivileges, PLSAPR_PRIVILEGE_SET Privileges) { - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; + PLSA_DB_OBJECT AccountObject; + PPRIVILEGE_SET CurrentPrivileges = NULL; + PPRIVILEGE_SET NewPrivileges = NULL; + ULONG PrivilegeSetSize = 0; + ULONG PrivilegeCount; + ULONG i, j, k; + BOOL bFound; + NTSTATUS Status; + + TRACE("(%p %u %p)\n", AccountHandle, AllPrivileges, Privileges); + + /* */ + if ((AllPrivileges == FALSE && Privileges == NULL) || + (AllPrivileges == TRUE && Privileges != NULL)) + return STATUS_INVALID_PARAMETER; + + /* Validate the AccountHandle */ + Status = LsapValidateDbObject(AccountHandle, + LsaDbAccountObject, + ACCOUNT_ADJUST_PRIVILEGES, + &AccountObject); + if (!NT_SUCCESS(Status)) + { + ERR("LsapValidateDbObject returned 0x%08lx\n", Status); + return Status; + } + + if (AllPrivileges == TRUE) + { + /* Delete the Privilgs attribute */ + Status = LsapDeleteObjectAttribute(AccountObject, + L"Privilgs"); + if (Status == STATUS_OBJECT_NAME_NOT_FOUND) + Status = STATUS_SUCCESS; + } + else + { + /* Get the size of the Privilgs attribute */ + Status = LsapGetObjectAttribute(AccountObject, + L"Privilgs", + NULL, + &PrivilegeSetSize); + if (!NT_SUCCESS(Status)) + goto done; + + /* Succeed, if there is no privilege set to remove privileges from */ + if (PrivilegeSetSize == 0) + { + Status = STATUS_SUCCESS; + goto done; + } + + /* Allocate memory for the stored privilege set */ + CurrentPrivileges = MIDL_user_allocate(PrivilegeSetSize); + if (CurrentPrivileges == NULL) + return STATUS_NO_MEMORY; + + /* Get the current privilege set */ + Status = LsapGetObjectAttribute(AccountObject, + L"Privilgs", + CurrentPrivileges, + &PrivilegeSetSize); + if (!NT_SUCCESS(Status)) + { + TRACE("LsapGetObjectAttribute() failed (Status 0x%08lx)\n", Status); + goto done; + } + + PrivilegeCount = CurrentPrivileges->PrivilegeCount; + TRACE("Current privilege count: %lu\n", PrivilegeCount); + + /* Calculate the number of privileges in the new privilege set */ + for (i = 0; i < CurrentPrivileges->PrivilegeCount; i++) + { + for (j = 0; j < Privileges->PrivilegeCount; j++) + { + if (RtlEqualLuid(&(CurrentPrivileges->Privilege[i].Luid), + &(Privileges->Privilege[j].Luid))) + { + if (PrivilegeCount > 0) + PrivilegeCount--; + } + } + } + TRACE("New privilege count: %lu\n", PrivilegeCount); + + if (PrivilegeCount == 0) + { + /* Delete the Privilgs attribute */ + Status = LsapDeleteObjectAttribute(AccountObject, + L"Privilgs"); + if (Status == STATUS_OBJECT_NAME_NOT_FOUND) + Status = STATUS_SUCCESS; + } + else + { + /* Calculate the size of the new privilege set and allocate it */ + PrivilegeSetSize = sizeof(PRIVILEGE_SET) + + (PrivilegeCount - 1) * sizeof(LUID_AND_ATTRIBUTES); + NewPrivileges = MIDL_user_allocate(PrivilegeSetSize); + if (NewPrivileges == NULL) + { + Status = STATUS_NO_MEMORY; + goto done; + } + + /* Initialize the new privilege set */ + NewPrivileges->PrivilegeCount = PrivilegeCount; + NewPrivileges->Control = 0; + + /* Copy the privileges which are not to be removed */ + for (i = 0, k = 0; i < CurrentPrivileges->PrivilegeCount; i++) + { + bFound = FALSE; + for (j = 0; j < Privileges->PrivilegeCount; j++) + { + if (RtlEqualLuid(&(CurrentPrivileges->Privilege[i].Luid), + &(Privileges->Privilege[j].Luid))) + bFound = TRUE; + } + + if (bFound == FALSE) + { + /* Copy the privilege */ + RtlCopyLuidAndAttributesArray(1, + &(CurrentPrivileges->Privilege[i]), + &(NewPrivileges->Privilege[k])); + k++; + } + } + + /* Set the new privilege set */ + Status = LsapSetObjectAttribute(AccountObject, + L"Privilgs", + NewPrivileges, + PrivilegeSetSize); + } + } + +done: + if (CurrentPrivileges != NULL) + MIDL_user_free(CurrentPrivileges); + + if (NewPrivileges != NULL) + MIDL_user_free(NewPrivileges); + + return Status; } @@ -2247,8 +2392,10 @@ ACCESS_MASK DesiredAccess, LSAPR_HANDLE *PolicyHandle) { - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; + return LsarOpenPolicy(SystemName, + ObjectAttributes, + DesiredAccess, + PolicyHandle); } @@ -2790,121 +2937,4 @@ return STATUS_NOT_IMPLEMENTED; } - -/* Function 82 */ -NTSTATUS WINAPI CredrFindBestCredential( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 83 */ -NTSTATUS WINAPI LsarSetAuditPolicy( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 84 */ -NTSTATUS WINAPI LsarQueryAuditPolicy( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 85 */ -NTSTATUS WINAPI LsarEnumerateAuditPolicy( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 86 */ -NTSTATUS WINAPI LsarEnumerateAuditCategories( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 87 */ -NTSTATUS WINAPI LsarEnumerateAuditSubCategories( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 88 */ -NTSTATUS WINAPI LsarLookupAuditCategoryName( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 89 */ -NTSTATUS WINAPI LsarLookupAuditSubCategoryName( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 90 */ -NTSTATUS WINAPI LsarSetAuditSecurity( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 91 */ -NTSTATUS WINAPI LsarQueryAuditSecurity( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 92 */ -NTSTATUS WINAPI CredReadByTokenHandle( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 93 */ -NTSTATUS WINAPI CredrRestoreCredentials( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - - -/* Function 94 */ -NTSTATUS WINAPI CredrBackupCredentials( - handle_t hBinding) -{ - UNIMPLEMENTED; - return STATUS_NOT_IMPLEMENTED; -} - /* EOF */ Modified: trunk/reactos/dll/win32/lsasrv/lsasrv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/lsasrv/lsasrv.h?… ============================================================================== --- trunk/reactos/dll/win32/lsasrv/lsasrv.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/lsasrv/lsasrv.h [iso-8859-1] Sat Dec 15 11:31:04 2012 @@ -124,6 +124,10 @@ LPVOID AttributeData, ULONG AttributeSize); +NTSTATUS +LsapDeleteObjectAttribute(PLSA_DB_OBJECT DbObject, + LPWSTR AttributeName); + /* lookup.c */ NTSTATUS LsapInitSids(VOID);

1 0

[akhaldi] 57919: [CMAKE] * Transition phase for asm handling between the actual and the upcoming RosBE.
by akhaldi＠svn.reactos.org 15 Dec '12

15 Dec '12

Author: akhaldi Date: Sat Dec 15 11:03:50 2012 New Revision: 57919 URL: http://svn.reactos.org/svn/reactos?rev=57919&view=rev Log: [CMAKE] * Transition phase for asm handling between the actual and the upcoming RosBE. Modified: trunk/reactos/cmake/CMakeDetermineASMCompiler.cmake Modified: trunk/reactos/cmake/CMakeDetermineASMCompiler.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/CMakeDetermineASMCom… ============================================================================== --- trunk/reactos/cmake/CMakeDetermineASMCompiler.cmake [iso-8859-1] (original) +++ trunk/reactos/cmake/CMakeDetermineASMCompiler.cmake [iso-8859-1] Sat Dec 15 11:03:50 2012 @@ -11,6 +11,8 @@ #============================================================================= # (To distribute this file outside of CMake, substitute the full # License text for the above reference.) + +if(CMAKE_VERSION VERSION_LESS 2.8.10) # determine the compiler to use for ASM programs @@ -180,3 +182,165 @@ SET(_CMAKE_ASM_COMPILER) SET(_CMAKE_ASM_COMPILER_ARG1) SET(_CMAKE_ASM_COMPILER_ENV_VAR) + +else() + +# determine the compiler to use for ASM programs + +include(${CMAKE_ROOT}/Modules/CMakeDetermineCompiler.cmake) + +if(NOT CMAKE_ASM${ASM_DIALECT}_COMPILER) + # prefer the environment variable ASM + if($ENV{ASM${ASM_DIALECT}} MATCHES ".+") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_INIT "$ENV{ASM${ASM_DIALECT}}") + endif() + + # finally list compilers to try + if("ASM${ASM_DIALECT}" STREQUAL "ASM") # the generic assembler support + if(NOT CMAKE_ASM_COMPILER_INIT) + if(CMAKE_C_COMPILER) + set(CMAKE_ASM_COMPILER "${CMAKE_C_COMPILER}" CACHE FILEPATH "The ASM compiler") + set(CMAKE_ASM_COMPILER_ID "${CMAKE_C_COMPILER_ID}") + elseif(CMAKE_CXX_COMPILER) + set(CMAKE_ASM_COMPILER "${CMAKE_CXX_COMPILER}" CACHE FILEPATH "The ASM compiler") + set(CMAKE_ASM_COMPILER_ID "${CMAKE_CXX_COMPILER_ID}") + else() + # List all default C and CXX compilers + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_LIST + ${_CMAKE_TOOLCHAIN_PREFIX}cc ${_CMAKE_TOOLCHAIN_PREFIX}gcc cl bcc xlc + CC ${_CMAKE_TOOLCHAIN_PREFIX}c++ ${_CMAKE_TOOLCHAIN_PREFIX}g++ aCC cl bcc xlC) + endif() + endif() + else() # some specific assembler "dialect" + if(NOT CMAKE_ASM${ASM_DIALECT}_COMPILER_INIT) + message(FATAL_ERROR "CMAKE_ASM${ASM_DIALECT}_COMPILER_INIT must be preset !") + endif() + endif() + + # Find the compiler. + _cmake_find_compiler(ASM${ASM_DIALECT}) + +else() + + # we only get here if CMAKE_ASM${ASM_DIALECT}_COMPILER was specified using -D or a pre-made CMakeCache.txt + # (e.g. via ctest) or set in CMAKE_TOOLCHAIN_FILE + # + # if a compiler was specified by the user but without path, + # now try to find it with the full path + # if it is found, force it into the cache, + # if not, don't overwrite the setting (which was given by the user) with "NOTFOUND" + get_filename_component(_CMAKE_USER_ASM${ASM_DIALECT}_COMPILER_PATH "${CMAKE_ASM${ASM_DIALECT}_COMPILER}" PATH) + if(NOT _CMAKE_USER_ASM${ASM_DIALECT}_COMPILER_PATH) + find_program(CMAKE_ASM${ASM_DIALECT}_COMPILER_WITH_PATH NAMES ${CMAKE_ASM${ASM_DIALECT}_COMPILER}) + mark_as_advanced(CMAKE_ASM${ASM_DIALECT}_COMPILER_WITH_PATH) + if(CMAKE_ASM${ASM_DIALECT}_COMPILER_WITH_PATH) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER ${CMAKE_ASM${ASM_DIALECT}_COMPILER_WITH_PATH} CACHE FILEPATH "Assembler" FORCE) + endif() + endif() +endif() +mark_as_advanced(CMAKE_ASM${ASM_DIALECT}_COMPILER) + +if (NOT _CMAKE_TOOLCHAIN_LOCATION) + get_filename_component(_CMAKE_TOOLCHAIN_LOCATION "${CMAKE_ASM${ASM_DIALECT}_COMPILER}" PATH) +endif () + + +if(NOT CMAKE_ASM${ASM_DIALECT}_COMPILER_ID) + + # Table of per-vendor compiler id flags with expected output. + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS GNU ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_GNU "--version") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_GNU "(GNU assembler)|(GCC)|(Free Software Foundation)") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS HP ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_HP "-V") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_HP "HP C") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS Intel ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_Intel "--version") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_Intel "(ICC)") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS SunPro ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_SunPro "-V") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_SunPro "Sun C") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS XL ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_XL "-qversion") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_XL "XL C") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS MSVC ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_MSVC "/?") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_MSVC "Microsoft") + + list(APPEND CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDORS TI_DSP ) + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_FLAGS_TI_DSP "-h") + set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID_VENDOR_REGEX_TI_DSP "Texas Instruments") + + include(CMakeDetermineCompilerId) + CMAKE_DETERMINE_COMPILER_ID_VENDOR(ASM${ASM_DIALECT}) + +endif() + +if(CMAKE_ASM${ASM_DIALECT}_COMPILER_ID) + message(STATUS "The ASM${ASM_DIALECT} compiler identification is ${CMAKE_ASM${ASM_DIALECT}_COMPILER_ID}") +else() + message(STATUS "The ASM${ASM_DIALECT} compiler identification is unknown") +endif() + + + +# If we have a gas/as cross compiler, they have usually some prefix, like +# e.g. powerpc-linux-gas, arm-elf-gas or i586-mingw32msvc-gas , optionally +# with a 3-component version number at the end +# The other tools of the toolchain usually have the same prefix +# NAME_WE cannot be used since then this test will fail for names lile +# "arm-unknown-nto-qnx6.3.0-gas.exe", where BASENAME would be +# "arm-unknown-nto-qnx6" instead of the correct "arm-unknown-nto-qnx6.3.0-" +if (NOT _CMAKE_TOOLCHAIN_PREFIX) + get_filename_component(COMPILER_BASENAME "${CMAKE_ASM${ASM_DIALECT}_COMPILER}" NAME) + if (COMPILER_BASENAME MATCHES "^(.+-)g?as(-[0-9]+\\.[0-9]+\\.[0-9]+)?(\\.exe)?$") + set(_CMAKE_TOOLCHAIN_PREFIX ${CMAKE_MATCH_1}) + endif () +endif () + +# Now try the C compiler regexp: +if (NOT _CMAKE_TOOLCHAIN_PREFIX) + if (COMPILER_BASENAME MATCHES "^(.+-)g?cc(-[0-9]+\\.[0-9]+\\.[0-9]+)?(\\.exe)?$") + set(_CMAKE_TOOLCHAIN_PREFIX ${CMAKE_MATCH_1}) + endif () +endif () + +# Finally try the CXX compiler regexp: +if (NOT _CMAKE_TOOLCHAIN_PREFIX) + if (COMPILER_BASENAME MATCHES "^(.+-)[gc]\\+\\+(-[0-9]+\\.[0-9]+\\.[0-9]+)?(\\.exe)?$") + set(_CMAKE_TOOLCHAIN_PREFIX ${CMAKE_MATCH_1}) + endif () +endif () + + +include(CMakeFindBinUtils) + +set(CMAKE_ASM${ASM_DIALECT}_COMPILER_ENV_VAR "ASM${ASM_DIALECT}") + +if(CMAKE_ASM${ASM_DIALECT}_COMPILER) + message(STATUS "Found assembler: ${CMAKE_ASM${ASM_DIALECT}_COMPILER}") +else() + message(STATUS "Didn't find assembler") +endif() + + +set(_CMAKE_ASM_COMPILER "${CMAKE_ASM${ASM_DIALECT}_COMPILER}") +set(_CMAKE_ASM_COMPILER_ID "${CMAKE_ASM${ASM_DIALECT}_COMPILER_ID}") +set(_CMAKE_ASM_COMPILER_ARG1 "${CMAKE_ASM${ASM_DIALECT}_COMPILER_ARG1}") +set(_CMAKE_ASM_COMPILER_ENV_VAR "${CMAKE_ASM${ASM_DIALECT}_COMPILER_ENV_VAR}") + +# configure variables set in this file for fast reload later on +configure_file(${CMAKE_ROOT}/Modules/CMakeASMCompiler.cmake.in + ${CMAKE_PLATFORM_INFO_DIR}/CMakeASM${ASM_DIALECT}Compiler.cmake IMMEDIATE @ONLY) + +set(_CMAKE_ASM_COMPILER) +set(_CMAKE_ASM_COMPILER_ARG1) +set(_CMAKE_ASM_COMPILER_ENV_VAR) + +endif() +

1 0

[hbelusca] 57918: [SHUTDOWN] Final attempt to fix resource compilation with MSVC. Seriously, their resource compiler sucks.
by hbelusca＠svn.reactos.org 15 Dec '12

15 Dec '12

Author: hbelusca Date: Sat Dec 15 01:20:06 2012 New Revision: 57918 URL: http://svn.reactos.org/svn/reactos?rev=57918&view=rev Log: [SHUTDOWN] Final attempt to fix resource compilation with MSVC. Seriously, their resource compiler sucks. Modified: trunk/reactos/base/applications/shutdown/lang/bg-BG.rc trunk/reactos/base/applications/shutdown/lang/de-DE.rc trunk/reactos/base/applications/shutdown/lang/el-GR.rc trunk/reactos/base/applications/shutdown/lang/en-US.rc trunk/reactos/base/applications/shutdown/lang/es-ES.rc trunk/reactos/base/applications/shutdown/lang/fr-FR.rc trunk/reactos/base/applications/shutdown/lang/it-IT.rc trunk/reactos/base/applications/shutdown/lang/ja-JP.rc trunk/reactos/base/applications/shutdown/lang/ko-KR.rc trunk/reactos/base/applications/shutdown/lang/lt-LT.rc trunk/reactos/base/applications/shutdown/lang/nl-NL.rc trunk/reactos/base/applications/shutdown/lang/no-NO.rc trunk/reactos/base/applications/shutdown/lang/pl-PL.rc trunk/reactos/base/applications/shutdown/lang/pt-BR.rc trunk/reactos/base/applications/shutdown/lang/ro-RO.rc trunk/reactos/base/applications/shutdown/lang/ru-RU.rc trunk/reactos/base/applications/shutdown/lang/sk-SK.rc trunk/reactos/base/applications/shutdown/lang/sv-SE.rc trunk/reactos/base/applications/shutdown/lang/uk-UA.rc trunk/reactos/base/applications/shutdown/lang/zh-CN.rc trunk/reactos/base/applications/shutdown/lang/zh-TW.rc [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/base/applications/shutdown/lang/bg-BG.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/el-GR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/it-IT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ko-KR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/lt-LT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/nl-NL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pt-BR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ro-RO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sv-SE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-CN.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-TW.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown…

1 0

[hbelusca] 57917: [SHUTDOWN] - Fix a signed / unsigned warning. - Attempt to fix resource compilation with MSVC.
by hbelusca＠svn.reactos.org 15 Dec '12

15 Dec '12

Author: hbelusca Date: Sat Dec 15 00:41:16 2012 New Revision: 57917 URL: http://svn.reactos.org/svn/reactos?rev=57917&view=rev Log: [SHUTDOWN] - Fix a signed / unsigned warning. - Attempt to fix resource compilation with MSVC. Modified: trunk/reactos/base/applications/shutdown/lang/bg-BG.rc trunk/reactos/base/applications/shutdown/lang/de-DE.rc trunk/reactos/base/applications/shutdown/lang/el-GR.rc trunk/reactos/base/applications/shutdown/lang/en-US.rc trunk/reactos/base/applications/shutdown/lang/es-ES.rc trunk/reactos/base/applications/shutdown/lang/fr-FR.rc trunk/reactos/base/applications/shutdown/lang/it-IT.rc trunk/reactos/base/applications/shutdown/lang/ja-JP.rc trunk/reactos/base/applications/shutdown/lang/ko-KR.rc trunk/reactos/base/applications/shutdown/lang/lt-LT.rc trunk/reactos/base/applications/shutdown/lang/nl-NL.rc trunk/reactos/base/applications/shutdown/lang/no-NO.rc trunk/reactos/base/applications/shutdown/lang/pl-PL.rc trunk/reactos/base/applications/shutdown/lang/pt-BR.rc trunk/reactos/base/applications/shutdown/lang/ro-RO.rc trunk/reactos/base/applications/shutdown/lang/ru-RU.rc trunk/reactos/base/applications/shutdown/lang/sk-SK.rc trunk/reactos/base/applications/shutdown/lang/sv-SE.rc trunk/reactos/base/applications/shutdown/lang/uk-UA.rc trunk/reactos/base/applications/shutdown/lang/zh-CN.rc trunk/reactos/base/applications/shutdown/lang/zh-TW.rc trunk/reactos/base/applications/shutdown/shutdown.c [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/base/applications/shutdown/lang/bg-BG.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/el-GR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/it-IT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ko-KR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/lt-LT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/nl-NL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pt-BR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ro-RO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sv-SE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-CN.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-TW.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/shutdown.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown…

1 0

[hbelusca] 57916: [SHUTDOWN] Improve functionnality of the shutdown utility : * Added extra arguments based upon Windows 7. * Added options to CommandLineOptions function. * Simplify the PrintResou...
by hbelusca＠svn.reactos.org 14 Dec '12

14 Dec '12

Author: hbelusca Date: Fri Dec 14 23:29:22 2012 New Revision: 57916 URL: http://svn.reactos.org/svn/reactos?rev=57916&view=rev Log: [SHUTDOWN] Improve functionnality of the shutdown utility : * Added extra arguments based upon Windows 7. * Added options to CommandLineOptions function. * Simplify the PrintResourceString() function. * Removed PrintUsage() since PrintResourceString() does a more flexible job at printing strings out. * Added clear error messages when there is a problem. This makes it a bit easier to determine if there is a problem. * Convert all the C++ comments to C. * Since each version of Windows has a different upper limit for the length of the comment, a work around solution is used that first checks the length of the comment based upon the version of Windows that the user is currently using. * Added support for hibernation mode, but the function SetSuspendState() is only a stub right now. * In cases where the shutdown/restart operation is done remotely, now the proper permission is granted if the user has the proper permission. * Added the REASON structure, which contains all the reason code combinations required for giving the proper reason code to the computer. * Added bare bones basic support for the GUI option (/i). However, until the required features are added to ReactOS, the GUI won't do anything. * Convert the program to full UNICODE support. CORE-6613 #resolve #comment Committed in r57916. Thanks :) Added: trunk/reactos/base/applications/shutdown/gui.c (with props) Modified: trunk/reactos/base/applications/shutdown/CMakeLists.txt trunk/reactos/base/applications/shutdown/lang/bg-BG.rc trunk/reactos/base/applications/shutdown/lang/de-DE.rc trunk/reactos/base/applications/shutdown/lang/el-GR.rc trunk/reactos/base/applications/shutdown/lang/en-US.rc trunk/reactos/base/applications/shutdown/lang/es-ES.rc trunk/reactos/base/applications/shutdown/lang/fr-FR.rc trunk/reactos/base/applications/shutdown/lang/it-IT.rc trunk/reactos/base/applications/shutdown/lang/ja-JP.rc trunk/reactos/base/applications/shutdown/lang/ko-KR.rc trunk/reactos/base/applications/shutdown/lang/lt-LT.rc trunk/reactos/base/applications/shutdown/lang/nl-NL.rc trunk/reactos/base/applications/shutdown/lang/no-NO.rc trunk/reactos/base/applications/shutdown/lang/pl-PL.rc trunk/reactos/base/applications/shutdown/lang/pt-BR.rc trunk/reactos/base/applications/shutdown/lang/ro-RO.rc trunk/reactos/base/applications/shutdown/lang/ru-RU.rc trunk/reactos/base/applications/shutdown/lang/sk-SK.rc trunk/reactos/base/applications/shutdown/lang/sv-SE.rc trunk/reactos/base/applications/shutdown/lang/uk-UA.rc trunk/reactos/base/applications/shutdown/lang/zh-CN.rc trunk/reactos/base/applications/shutdown/lang/zh-TW.rc trunk/reactos/base/applications/shutdown/misc.c trunk/reactos/base/applications/shutdown/precomp.h trunk/reactos/base/applications/shutdown/resource.h trunk/reactos/base/applications/shutdown/rsrc.rc trunk/reactos/base/applications/shutdown/shutdown.c (contents, props changed) trunk/reactos/base/applications/shutdown/shutdown.rc [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/base/applications/shutdown/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Added: trunk/reactos/base/applications/shutdown/gui.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/bg-BG.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/el-GR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/it-IT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ko-KR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/lt-LT.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/nl-NL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/pt-BR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ro-RO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/sv-SE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-CN.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/lang/zh-TW.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/misc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/precomp.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/resource.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/rsrc.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/shutdown.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown… Modified: trunk/reactos/base/applications/shutdown/shutdown.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/shutdown…

1 0

[akhaldi] 57915: [COMCTL32_WINETEST] * Sync with Wine 1.5.19.
by akhaldi＠svn.reactos.org 14 Dec '12

14 Dec '12

Author: akhaldi Date: Fri Dec 14 23:24:10 2012 New Revision: 57915 URL: http://svn.reactos.org/svn/reactos?rev=57915&view=rev Log: [COMCTL32_WINETEST] * Sync with Wine 1.5.19. Added: trunk/rostests/winetests/comctl32/pager.c (with props) trunk/rostests/winetests/comctl32/syslink.c (with props) Modified: trunk/rostests/winetests/comctl32/CMakeLists.txt trunk/rostests/winetests/comctl32/comboex.c trunk/rostests/winetests/comctl32/datetime.c trunk/rostests/winetests/comctl32/imagelist.c trunk/rostests/winetests/comctl32/listview.c trunk/rostests/winetests/comctl32/monthcal.c trunk/rostests/winetests/comctl32/mru.c trunk/rostests/winetests/comctl32/propsheet.c trunk/rostests/winetests/comctl32/resources.h trunk/rostests/winetests/comctl32/rsrc.rc trunk/rostests/winetests/comctl32/status.c trunk/rostests/winetests/comctl32/tab.c trunk/rostests/winetests/comctl32/testlist.c trunk/rostests/winetests/comctl32/toolbar.c trunk/rostests/winetests/comctl32/tooltips.c trunk/rostests/winetests/comctl32/trackbar.c trunk/rostests/winetests/comctl32/treeview.c Modified: trunk/rostests/winetests/comctl32/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/CMakeL… ============================================================================== --- trunk/rostests/winetests/comctl32/CMakeLists.txt [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/CMakeLists.txt [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -14,11 +14,13 @@ misc.c monthcal.c mru.c + pager.c progress.c propsheet.c rebar.c status.c subclass.c + syslink.c tab.c toolbar.c tooltips.c Modified: trunk/rostests/winetests/comctl32/comboex.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/comboe… ============================================================================== --- trunk/rostests/winetests/comctl32/comboex.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/comboex.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -345,6 +345,11 @@ "Current Selection: expected %d, got %d\n", 4, idx); ok(received_end_edit, "Expected to receive a CBEN_ENDEDIT message\n"); + SetFocus( hComboExParentWnd ); + ok( GetFocus() == hComboExParentWnd, "got %p\n", GetFocus() ); + SetFocus( hComboEx ); + ok( GetFocus() == hEdit, "got %p\n", GetFocus() ); + DestroyWindow(hComboEx); } Modified: trunk/rostests/winetests/comctl32/datetime.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/dateti… ============================================================================== --- trunk/rostests/winetests/comctl32/datetime.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/datetime.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -121,6 +121,18 @@ { 0 } }; +static const struct message test_dtm_set_and_get_systime_with_limits[] = { + { DTM_SETRANGE, sent|wparam, GDTR_MIN | GDTR_MAX }, + { DTM_GETRANGE, sent|wparam, 0 }, + { DTM_SETSYSTEMTIME, sent|wparam, 0 }, + { DTM_GETSYSTEMTIME, sent|wparam, 0 }, + { DTM_SETSYSTEMTIME, sent|wparam, 0 }, + { DTM_GETSYSTEMTIME, sent|wparam, 0 }, + { DTM_SETSYSTEMTIME, sent|wparam, 0 }, + { DTM_GETSYSTEMTIME, sent|wparam, 0 }, + { 0 } +}; + static LRESULT WINAPI datetime_subclass_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) { WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); @@ -660,6 +672,55 @@ expect(4, (LRESULT)getSt.wDayOfWeek); st.wDayOfWeek = 4; expect_systime(&st, &getSt); +} + +static void test_dtm_set_and_get_systemtime_with_limits(void) +{ + LRESULT r; + SYSTEMTIME st[2], getSt[2], refSt; + HWND hWnd; + + hWnd = create_datetime_control(DTS_SHOWNONE); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + + /* set range */ + fill_systime_struct(&st[0], 1980, 1, 3, 23, 14, 34, 37, 465); + fill_systime_struct(&st[1], 2007, 3, 2, 31, 23, 59, 59, 999); + + r = SendMessage(hWnd, DTM_SETRANGE, GDTR_MIN | GDTR_MAX, (LPARAM)st); + expect(1, r); + r = SendMessage(hWnd, DTM_GETRANGE, 0, (LPARAM)getSt); + ok(r == (GDTR_MIN | GDTR_MAX), "Expected %x, not %x(GDTR_MIN) or %x(GDTR_MAX), got %lx\n", (GDTR_MIN | GDTR_MAX), GDTR_MIN, GDTR_MAX, r); + expect_systime(&st[0], &getSt[0]); + expect_systime(&st[1], &getSt[1]); + + /* Initially set a valid time */ + fill_systime_struct(&refSt, 1999, 9, 4, 9, 19, 9, 9, 999); + r = SendMessage(hWnd, DTM_SETSYSTEMTIME, GDT_VALID, (LPARAM)&refSt); + expect(1, r); + r = SendMessage(hWnd, DTM_GETSYSTEMTIME, 0, (LPARAM)&getSt[0]); + ok(r == GDT_VALID, "Expected %d, not %d(GDT_NONE) or %d(GDT_ERROR), got %ld\n", GDT_VALID, GDT_NONE, GDT_ERROR, r); + expect_systime(&refSt, &getSt[0]); + + /* Now set an out-of-bounds time */ + fill_systime_struct(&st[0], 2010, 1, 0, 1, 0, 0, 0, 0); + + r = SendMessage(hWnd, DTM_SETSYSTEMTIME, GDT_VALID, (LPARAM)&st[0]); + expect(1, r); + r = SendMessage(hWnd, DTM_GETSYSTEMTIME, 0, (LPARAM)&getSt[0]); + ok(r == GDT_VALID, "Expected %d, not %d(GDT_NONE) or %d(GDT_ERROR), got %ld\n", GDT_VALID, GDT_NONE, GDT_ERROR, r); + expect_systime(&refSt, &getSt[0]); + + fill_systime_struct(&st[0], 1977, 1, 0, 1, 0, 0, 0, 0); + + r = SendMessage(hWnd, DTM_SETSYSTEMTIME, GDT_VALID, (LPARAM)&st[0]); + expect(1, r); + r = SendMessage(hWnd, DTM_GETSYSTEMTIME, 0, (LPARAM)&getSt[0]); + ok(r == GDT_VALID, "Expected %d, not %d(GDT_NONE) or %d(GDT_ERROR), got %ld\n", GDT_VALID, GDT_NONE, GDT_ERROR, r); + expect_systime(&refSt, &getSt[0]); + + ok_sequence(sequences, DATETIME_SEQ_INDEX, test_dtm_set_and_get_systime_with_limits, "test_dtm_set_and_get_systime_with_limits", FALSE); DestroyWindow(hWnd); } @@ -749,6 +810,7 @@ test_dtm_set_and_get_range(); test_dtm_set_range_swap_min_max(); test_dtm_set_and_get_system_time(); + test_dtm_set_and_get_systemtime_with_limits(); test_wm_set_get_text(); test_dts_shownone(); } Modified: trunk/rostests/winetests/comctl32/imagelist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/imagel… ============================================================================== --- trunk/rostests/winetests/comctl32/imagelist.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/imagelist.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -619,7 +619,7 @@ else my_is->iml_data = HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, my_is->iml_data, my_is->iml_data_size); - return my_is->iml_data ? TRUE : FALSE; + return my_is->iml_data != NULL; } static HRESULT STDMETHODCALLTYPE Test_Stream_Write(IStream *iface, const void *pv, ULONG cb, @@ -1423,7 +1423,7 @@ /* test reference counting on destruction */ imgl = (IImageList*)createImageList(32, 32); - ret = IUnknown_AddRef(imgl); + ret = IImageList_AddRef(imgl); ok(ret == 2, "Expected 2, got %d\n", ret); ret = ImageList_Destroy((HIMAGELIST)imgl); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); @@ -1433,7 +1433,7 @@ ok(ret == FALSE, "Expected FALSE, got %d\n", ret); imgl = (IImageList*)createImageList(32, 32); - ret = IUnknown_AddRef(imgl); + ret = IImageList_AddRef(imgl); ok(ret == 2, "Expected 2, got %d\n", ret); ret = ImageList_Destroy((HIMAGELIST)imgl); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); Modified: trunk/rostests/winetests/comctl32/listview.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/listvi… ============================================================================== --- trunk/rostests/winetests/comctl32/listview.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/listview.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -3,7 +3,7 @@ * * Copyright 2006 Mike McCormack for CodeWeavers * Copyright 2007 George Gov - * Copyright 2009-2011 Nikolay Sivov + * Copyright 2009-2012 Nikolay Sivov * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -50,11 +50,11 @@ static BOOL blockEdit; /* return nonzero on NM_HOVER */ static BOOL g_block_hover; -/* dumps LVN_ITEMCHANGED message data */ -static BOOL g_dump_itemchanged; +/* notification data for LVN_ITEMCHANGED */ +static NMLISTVIEW g_nmlistview; /* format reported to control: -1 falls to defproc, anything else returned */ -static INT notifyFormat; +static INT notifyFormat; /* indicates we're running < 5.80 version */ static BOOL g_is_below_5; /* item data passed to LVN_GETDISPINFOA */ @@ -391,11 +391,9 @@ } break; case LVN_ITEMCHANGED: - if (g_dump_itemchanged) { NMLISTVIEW *nmlv = (NMLISTVIEW*)lParam; - trace("LVN_ITEMCHANGED: item=%d,new=%x,old=%x,changed=%x\n", - nmlv->iItem, nmlv->uNewState, nmlv->uOldState, nmlv->uChanged); + g_nmlistview = *nmlv; } break; case LVN_GETDISPINFOA: @@ -2092,7 +2090,6 @@ { "using VK_HOME", -1, VK_HOME, 1, -1 } }; - hwnd = create_listview_control(LVS_REPORT); for (i=0;i<items;i++) { @@ -2239,6 +2236,131 @@ r = ListView_GetSelectedCount(hwnd); expect(0, r); + /* 1. selection mark is update when new focused item is set */ + style = GetWindowLongPtrA(hwnd, GWL_STYLE); + SetWindowLongPtrA(hwnd, GWL_STYLE, style & ~LVS_SINGLESEL); + + r = SendMessage(hwnd, LVM_SETSELECTIONMARK, 0, -1); + expect(-1, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + r = SendMessage(hwnd, LVM_SETITEMSTATE, 0, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(0, r); + + /* it's not updated if already set */ + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + r = SendMessage(hwnd, LVM_SETITEMSTATE, 1, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(0, r); + + r = SendMessage(hwnd, LVM_SETSELECTIONMARK, 0, -1); + expect(0, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + r = SendMessage(hwnd, LVM_SETITEMSTATE, 1, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(-1, r); + + /* need to reset focused item first */ + item.stateMask = LVIS_FOCUSED; + item.state = 0; + r = SendMessage(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); + expect(TRUE, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + r = SendMessage(hwnd, LVM_SETITEMSTATE, 2, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(2, r); + + item.stateMask = LVIS_FOCUSED; + item.state = 0; + r = SendMessage(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(2, r); + + /* 2. same tests, with LVM_SETITEM */ + style = GetWindowLongPtrA(hwnd, GWL_STYLE); + SetWindowLongPtrA(hwnd, GWL_STYLE, style & ~LVS_SINGLESEL); + + r = SendMessage(hwnd, LVM_SETSELECTIONMARK, 0, -1); + expect(2, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + item.mask = LVIF_STATE; + item.iItem = item.iSubItem = 0; + r = SendMessage(hwnd, LVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(0, r); + + /* it's not updated if already set */ + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + item.mask = LVIF_STATE; + item.iItem = 1; + item.iSubItem = 0; + r = SendMessage(hwnd, LVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(0, r); + + r = SendMessage(hwnd, LVM_SETSELECTIONMARK, 0, -1); + expect(0, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + item.mask = LVIF_STATE; + item.iItem = 1; + item.iSubItem = 0; + r = SendMessage(hwnd, LVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(-1, r); + + /* need to reset focused item first */ + item.stateMask = LVIS_FOCUSED; + item.state = 0; + r = SendMessage(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); + expect(TRUE, r); + + item.stateMask = LVIS_FOCUSED; + item.state = LVIS_FOCUSED; + item.mask = LVIF_STATE; + item.iItem = 2; + item.iSubItem = 0; + r = SendMessage(hwnd, LVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(2, r); + + item.stateMask = LVIS_FOCUSED; + item.state = 0; + r = SendMessage(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); + expect(TRUE, r); + + r = SendMessage(hwnd, LVM_GETSELECTIONMARK, 0, 0); + expect(2, r); + DestroyWindow(hwnd); } @@ -2629,6 +2751,8 @@ static void test_ownerdata(void) { + static char test_str[] = "test"; + HWND hwnd; LONG_PTR style, ret; DWORD res; @@ -2713,7 +2837,7 @@ expect(1, res); DestroyWindow(hwnd); - /* LVM_SETITEM is unsupported on LVS_OWNERDATA */ + /* LVM_SETITEM and LVM_SETITEMTEXT is unsupported on LVS_OWNERDATA */ hwnd = create_listview_control(LVS_OWNERDATA | LVS_REPORT); ok(hwnd != NULL, "failed to create a listview window\n"); res = SendMessageA(hwnd, LVM_SETITEMCOUNT, 1, 0); @@ -2727,6 +2851,10 @@ item.state = LVIS_SELECTED; res = SendMessageA(hwnd, LVM_SETITEM, 0, (LPARAM)&item); expect(FALSE, res); + memset(&item, 0, sizeof(item)); + item.pszText = test_str; + res = SendMessageA(hwnd, LVM_SETITEMTEXT, 0, (LPARAM)&item); + expect(FALSE, res); DestroyWindow(hwnd); /* check notifications after focused/selected changed */ @@ -2768,34 +2896,67 @@ item.stateMask = LVIS_SELECTED; item.state = LVIS_SELECTED; - g_dump_itemchanged = TRUE; + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == LVIS_SELECTED, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == 0, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_setstate_all_parent_seq, - "ownerdata select all notification", TRUE); + "ownerdata select all notification", FALSE); /* select all again, note that all items are selected already */ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_SELECTED; item.state = LVIS_SELECTED; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == LVIS_SELECTED, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == 0, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_setstate_all_parent_seq, - "ownerdata select all notification", TRUE); + "ownerdata select all notification", FALSE); + /* deselect all */ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_SELECTED; item.state = 0; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == 0, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == LVIS_SELECTED, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_deselect_all_parent_seq, "ownerdata deselect all notification", TRUE); + + /* nothing selected, deselect all again */ + flush_sequences(sequences, NUM_MSG_SEQUENCES); + item.stateMask = LVIS_SELECTED; + item.state = 0; + + res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); + expect(TRUE, res); + + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "ownerdata deselect all notification", TRUE); /* select one, then deselect all */ item.stateMask = LVIS_SELECTED; @@ -2805,10 +2966,18 @@ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_SELECTED; item.state = 0; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == 0, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == LVIS_SELECTED, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_deselect_all_parent_seq, "ownerdata select all notification", TRUE); @@ -2824,16 +2993,18 @@ item.stateMask = LVIS_FOCUSED; res = SendMessageA(hwnd, LVM_GETITEMSTATE, 0, LVIS_FOCUSED); expect(0, res); + /* setting all to focused returns failure value */ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_FOCUSED; item.state = LVIS_FOCUSED; - g_dump_itemchanged = TRUE; + res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(FALSE, res); - g_dump_itemchanged = FALSE; + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "ownerdata focus all notification", FALSE); + /* focus single item, remove all */ item.stateMask = LVIS_FOCUSED; item.state = LVIS_FOCUSED; @@ -2842,32 +3013,58 @@ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_FOCUSED; item.state = 0; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == 0, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == LVIS_FOCUSED, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_defocus_all_parent_seq, "ownerdata remove focus all notification", TRUE); + /* set all cut */ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_CUT; item.state = LVIS_CUT; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == LVIS_CUT, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == 0, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_setstate_all_parent_seq, - "ownerdata cut all notification", TRUE); + "ownerdata cut all notification", FALSE); + /* all marked cut, try again */ flush_sequences(sequences, NUM_MSG_SEQUENCES); item.stateMask = LVIS_CUT; item.state = LVIS_CUT; - g_dump_itemchanged = TRUE; + + memset(&g_nmlistview, 0xcc, sizeof(g_nmlistview)); res = SendMessageA(hwnd, LVM_SETITEMSTATE, -1, (LPARAM)&item); expect(TRUE, res); - g_dump_itemchanged = FALSE; + ok(g_nmlistview.iItem == -1, "got item %d\n", g_nmlistview.iItem); + ok(g_nmlistview.iSubItem == 0, "got subitem %d\n", g_nmlistview.iSubItem); + ok(g_nmlistview.uNewState == LVIS_CUT, "got new state 0x%08x\n", g_nmlistview.uNewState); + ok(g_nmlistview.uOldState == 0, "got old state 0x%08x\n", g_nmlistview.uOldState); + ok(g_nmlistview.uChanged == LVIF_STATE, "got changed 0x%08x\n", g_nmlistview.uChanged); + ok(g_nmlistview.ptAction.x == 0 && g_nmlistview.ptAction.y == 0, "got wrong ptAction value\n"); + ok(g_nmlistview.lParam == 0, "got wrong lparam\n"); + ok_sequence(sequences, PARENT_SEQ_INDEX, ownerdata_setstate_all_parent_seq, - "ownerdata cut all notification #2", TRUE); + "ownerdata cut all notification #2", FALSE); DestroyWindow(hwnd); Modified: trunk/rostests/winetests/comctl32/monthcal.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/monthc… ============================================================================== --- trunk/rostests/winetests/comctl32/monthcal.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/monthcal.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -40,6 +40,8 @@ #define NUM_MSG_SEQUENCES 2 #define PARENT_SEQ_INDEX 0 #define MONTHCAL_SEQ_INDEX 1 + +#define SEL_NOTIFY_TEST_ID 100 static struct msg_sequence *sequences[NUM_MSG_SEQUENCES]; @@ -385,6 +387,7 @@ /* set both limits, then set max < min */ GetSystemTime(&st[0]); + st[0].wDay = 25; st[1] = st[0]; st[1].wYear++; ok(SendMessage(hwnd, MCM_SETRANGE, GDTR_MIN|GDTR_MAX, (LPARAM)st), "Failed to set limits\n"); @@ -482,6 +485,39 @@ nmstate->prgDayState = months; + return TRUE; + } + case MCN_SELECT: + case MCN_SELCHANGE: + { + NMSELCHANGE *nmchg = (NMSELCHANGE*)lParam; + SYSTEMTIME st[2]; + BOOL is_multisel = GetWindowLongPtr(nmchg->nmhdr.hwndFrom, GWL_STYLE) & + MCS_MULTISELECT; + + if(GetWindowLongPtr(nmchg->nmhdr.hwndFrom, GWLP_ID) != SEL_NOTIFY_TEST_ID) + break; + SendMessage(nmchg->nmhdr.hwndFrom, is_multisel ? MCM_GETSELRANGE : + MCM_GETCURSEL, 0, (LPARAM)st); + + expect(st[0].wYear, nmchg->stSelStart.wYear); + expect(st[0].wMonth, nmchg->stSelStart.wMonth); + expect(0, nmchg->stSelStart.wDayOfWeek); + expect(st[0].wDay, nmchg->stSelStart.wDay); + + if(is_multisel) + { + expect(st[1].wYear, nmchg->stSelEnd.wYear); + expect(st[1].wMonth, nmchg->stSelEnd.wMonth); + expect(0, nmchg->stSelEnd.wDayOfWeek); + expect(st[1].wDay, nmchg->stSelEnd.wDay); + } + else + ok(!(nmchg->stSelEnd.wYear | nmchg->stSelEnd.wMonth | + nmchg->stSelEnd.wDayOfWeek | nmchg->stSelEnd.wDay | + nmchg->stSelEnd.wHour | nmchg->stSelEnd.wMinute | + nmchg->stSelEnd.wSecond | nmchg->stSelEnd.wMilliseconds), + "Non-zero member in stSelEnd\n"); return TRUE; } default: @@ -1918,6 +1954,51 @@ DestroyWindow(hwnd); } +static void test_sel_notify(void) +{ + typedef struct + { + DWORD val; + const char* name; + } Monthcal_style; + + HWND hwnd; + RECT rc; + MCHITTESTINFO mchit = {sizeof(MCHITTESTINFO)}; + SYSTEMTIME st; + Monthcal_style styles[] = { + {MCS_NOTODAY, "MCS_NOTODAY"}, + {MCS_NOTODAY | MCS_MULTISELECT, "MCS_NOTODAY | MCS_MULTISELECT"}, + {MCS_DAYSTATE, "MCS_DAYSTATE"}, + {MCS_DAYSTATE | MCS_MULTISELECT, "MCS_DAYSTATE | MCS_MULTISELECT"} + }; + int i; + + for(i = 0; i < sizeof styles / sizeof styles[0]; i++) + { + trace("%s\n", styles[i].name); + hwnd = create_monthcal_control(styles[i].val); + SetWindowLongPtr(hwnd, GWLP_ID, SEL_NOTIFY_TEST_ID); + assert(hwnd); + SendMessage(hwnd, MCM_GETMINREQRECT, 0, (LPARAM)&rc); + MoveWindow(hwnd, 0, 0, rc.right, rc.bottom, FALSE); + /* Simulate mouse click on some unselected day to generate + MCN_SELECT and MCN_SELCHANGE notifications */ + mchit.pt.x = rc.right / 2; + mchit.pt.y = rc.bottom / 2; + SendMessage(hwnd, MCM_HITTEST, 0, (LPARAM)&mchit); + SendMessage(hwnd, MCM_GETCURSEL, 0, (LPARAM)&st); + while(st.wDay == mchit.st.wDay) /* Ensure that mchit.pt points to unselected day */ + { + mchit.pt.y++; + SendMessage(hwnd, MCM_HITTEST, 0, (LPARAM)&mchit); + } + SendMessage(hwnd, WM_LBUTTONDOWN, 0, MAKELPARAM(mchit.pt.x, mchit.pt.y)); + SendMessage(hwnd, WM_LBUTTONUP, 0, MAKELPARAM(mchit.pt.x, mchit.pt.y)); + DestroyWindow(hwnd); + } +} + START_TEST(monthcal) { BOOL (WINAPI *pInitCommonControlsEx)(const INITCOMMONCONTROLSEX*); @@ -1961,6 +2042,7 @@ test_selrange(); test_killfocus(); test_daystate(); + test_sel_notify(); if (!load_v6_module(&ctx_cookie, &hCtx)) { Modified: trunk/rostests/winetests/comctl32/mru.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/mru.c?… ============================================================================== --- trunk/rostests/winetests/comctl32/mru.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/mru.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -45,7 +45,7 @@ UINT fFlags; HKEY hKey; LPCSTR lpszSubKey; - PROC lpfnCompare; + int (CALLBACK *lpfnCompare)(LPCSTR, LPCSTR); } MRUINFOA; typedef struct tagMRUINFOW @@ -55,7 +55,7 @@ UINT fFlags; HKEY hKey; LPCWSTR lpszSubKey; - PROC lpfnCompare; + int (CALLBACK *lpfnCompare)(LPCWSTR, LPCWSTR); } MRUINFOW; #define MRU_STRING 0 /* this one's invented */ @@ -221,7 +221,7 @@ } } -static INT CALLBACK cmp_mru_strA(LPCVOID data1, LPCVOID data2) +static int CALLBACK cmp_mru_strA(LPCSTR data1, LPCSTR data2) { return lstrcmpiA(data1, data2); } @@ -252,7 +252,7 @@ infoA.fFlags = MRU_STRING; infoA.hKey = NULL; infoA.lpszSubKey = REG_TEST_SUBKEYA; - infoA.lpfnCompare = (PROC)cmp_mru_strA; + infoA.lpfnCompare = cmp_mru_strA; SetLastError(0); hMRU = pCreateMRUListA(&infoA); @@ -266,7 +266,7 @@ infoA.fFlags = MRU_STRING; infoA.hKey = NULL; infoA.lpszSubKey = REG_TEST_SUBKEYA; - infoA.lpfnCompare = (PROC)cmp_mru_strA; + infoA.lpfnCompare = cmp_mru_strA; SetLastError(0); hMRU = pCreateMRUListA(&infoA); @@ -280,7 +280,7 @@ infoA.fFlags = MRU_STRING; infoA.hKey = NULL; infoA.lpszSubKey = REG_TEST_SUBKEYA; - infoA.lpfnCompare = (PROC)cmp_mru_strA; + infoA.lpfnCompare = cmp_mru_strA; SetLastError(0); hMRU = pCreateMRUListA(&infoA); @@ -294,7 +294,7 @@ infoA.fFlags = MRU_STRING; infoA.hKey = NULL; infoA.lpszSubKey = NULL; - infoA.lpfnCompare = (PROC)cmp_mru_strA; + infoA.lpfnCompare = cmp_mru_strA; SetLastError(0); hMRU = pCreateMRUListA(&infoA); @@ -312,7 +312,7 @@ infoA.fFlags = MRU_STRING; infoA.hKey = hKey; infoA.lpszSubKey = REG_TEST_SUBKEYA; - infoA.lpfnCompare = (PROC)cmp_mru_strA; + infoA.lpfnCompare = cmp_mru_strA; hMRU = pCreateMRUListA(&infoA); ok(hMRU && !GetLastError(), @@ -447,17 +447,7 @@ /* FreeMRUList(NULL) crashes on Win98 OSR0 */ } -/* -typedef struct tagMRUINFOA -{ - DWORD cbSize; - UINT uMax; - UINT fFlags; - HKEY hKey; - LPCSTR lpszSubKey; - PROC lpfnCompare; -} MRUINFOA; -*/ + typedef struct { MRUINFOA mruA; BOOL ret; Added: trunk/rostests/winetests/comctl32/pager.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/pager.… ============================================================================== --- trunk/rostests/winetests/comctl32/pager.c (added) +++ trunk/rostests/winetests/comctl32/pager.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -1,0 +1,204 @@ +/* + * Unit tests for the pager control + * + * Copyright 2012 Alexandre Julliard + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <windows.h> +#include <commctrl.h> + +#include "wine/test.h" +#include "msg.h" + +#define NUM_MSG_SEQUENCES 1 +#define PAGER_SEQ_INDEX 0 + +static HWND parent_wnd; + +static BOOL (WINAPI *pSetWindowSubclass)(HWND, SUBCLASSPROC, UINT_PTR, DWORD_PTR); + +static struct msg_sequence *sequences[NUM_MSG_SEQUENCES]; + +static const struct message set_child_seq[] = { + { PGM_SETCHILD, sent }, + { WM_WINDOWPOSCHANGING, sent }, + { WM_NCCALCSIZE, sent|wparam, TRUE }, + { WM_NOTIFY, sent|id|parent, 0, 0, PGN_CALCSIZE }, + { WM_WINDOWPOSCHANGED, sent }, + { 0 } +}; + +static const struct message set_pos_seq[] = { + { PGM_SETPOS, sent }, + { WM_WINDOWPOSCHANGING, sent }, + { WM_NCCALCSIZE, sent|wparam, TRUE }, + { WM_NOTIFY, sent|id|parent, 0, 0, PGN_CALCSIZE }, + { WM_WINDOWPOSCHANGED, sent }, + { WM_MOVE, sent|optional }, + { WM_SIZE, sent|optional }, + { 0 } +}; + +static const struct message set_pos_empty_seq[] = { + { PGM_SETPOS, sent }, + { 0 } +}; + +static LRESULT WINAPI parent_wnd_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + static LONG defwndproc_counter = 0; + LRESULT ret; + struct message msg; + + /* log system messages, except for painting */ + if (message < WM_USER && + message != WM_PAINT && + message != WM_ERASEBKGND && + message != WM_NCPAINT && + message != WM_NCHITTEST && + message != WM_GETTEXT && + message != WM_GETICON && + message != WM_DEVICECHANGE) + { + trace("parent: %p, %04x, %08lx, %08lx\n", hwnd, message, wParam, lParam); + + msg.message = message; + msg.flags = sent|wparam|lparam|parent; + if (defwndproc_counter) msg.flags |= defwinproc; + msg.wParam = wParam; + msg.lParam = lParam; + if (message == WM_NOTIFY && lParam) msg.id = ((NMHDR*)lParam)->code; + add_message(sequences, PAGER_SEQ_INDEX, &msg); + } + + defwndproc_counter++; + ret = DefWindowProcA(hwnd, message, wParam, lParam); + defwndproc_counter--; + + return ret; +} + +static BOOL register_parent_wnd_class(void) +{ + WNDCLASSA cls; + + cls.style = 0; + cls.lpfnWndProc = parent_wnd_proc; + cls.cbClsExtra = 0; + cls.cbWndExtra = 0; + cls.hInstance = GetModuleHandleA(NULL); + cls.hIcon = 0; + cls.hCursor = LoadCursorA(0, IDC_ARROW); + cls.hbrBackground = GetStockObject(WHITE_BRUSH); + cls.lpszMenuName = NULL; + cls.lpszClassName = "Pager test parent class"; + return RegisterClassA(&cls); +} + +static HWND create_parent_window(void) +{ + if (!register_parent_wnd_class()) + return NULL; + + return CreateWindow("Pager test parent class", "Pager test parent window", + WS_OVERLAPPED | WS_VISIBLE, + 0, 0, 200, 200, 0, NULL, GetModuleHandleA(NULL), NULL ); +} + +static LRESULT WINAPI pager_subclass_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); + struct message msg; + + trace("pager: %p, %04x, %08lx, %08lx\n", hwnd, message, wParam, lParam); + + msg.message = message; + msg.flags = sent|wparam|lparam; + msg.wParam = wParam; + msg.lParam = lParam; + msg.id = 0; + add_message(sequences, PAGER_SEQ_INDEX, &msg); + return CallWindowProcA(oldproc, hwnd, message, wParam, lParam); +} + +static HWND create_pager_control( DWORD style ) +{ + WNDPROC oldproc; + HWND hwnd; + RECT rect; + + GetClientRect( parent_wnd, &rect ); + hwnd = CreateWindowA( WC_PAGESCROLLERA, "pager", WS_CHILD | WS_BORDER | WS_VISIBLE | style, + 0, 0, 100, 100, parent_wnd, 0, GetModuleHandleA(0), 0 ); + oldproc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, (LONG_PTR)pager_subclass_proc); + SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); + return hwnd; +} + +static void test_pager(void) +{ + HWND pager, child; + RECT rect; + + pager = create_pager_control( PGS_HORZ ); + if (!pager) + { + win_skip( "Pager control not supported\n" ); + return; + } + child = CreateWindowA( "BUTTON", "button", WS_CHILD | WS_BORDER | WS_VISIBLE, 0, 0, 300, 300, + pager, 0, GetModuleHandleA(0), 0 ); + + flush_sequences( sequences, NUM_MSG_SEQUENCES ); + SendMessageA( pager, PGM_SETCHILD, 0, (LPARAM)child ); + ok_sequence(sequences, PAGER_SEQ_INDEX, set_child_seq, "set child", TRUE); + GetWindowRect( pager, &rect ); + ok( rect.right - rect.left == 100 && rect.bottom - rect.top == 100, + "pager resized %dx%d\n", rect.right - rect.left, rect.bottom - rect.top ); + + flush_sequences( sequences, NUM_MSG_SEQUENCES ); + SendMessageA( pager, PGM_SETPOS, 0, 10 ); + ok_sequence(sequences, PAGER_SEQ_INDEX, set_pos_seq, "set pos", TRUE); + GetWindowRect( pager, &rect ); + ok( rect.right - rect.left == 100 && rect.bottom - rect.top == 100, + "pager resized %dx%d\n", rect.right - rect.left, rect.bottom - rect.top ); + + flush_sequences( sequences, NUM_MSG_SEQUENCES ); + SendMessageA( pager, PGM_SETPOS, 0, 10 ); + ok_sequence(sequences, PAGER_SEQ_INDEX, set_pos_empty_seq, "set pos empty", TRUE); + + flush_sequences( sequences, NUM_MSG_SEQUENCES ); + SendMessageA( pager, PGM_SETPOS, 0, 9 ); + ok_sequence(sequences, PAGER_SEQ_INDEX, set_pos_seq, "set pos", TRUE); + + DestroyWindow( pager ); +} + +START_TEST(pager) +{ + HMODULE mod = GetModuleHandleA("comctl32.dll"); + + pSetWindowSubclass = (void*)GetProcAddress(mod, (LPSTR)410); + + InitCommonControls(); + init_msg_sequences(sequences, NUM_MSG_SEQUENCES); + + parent_wnd = create_parent_window(); + ok(parent_wnd != NULL, "Failed to create parent window!\n"); + + test_pager(); +} Propchange: trunk/rostests/winetests/comctl32/pager.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/rostests/winetests/comctl32/propsheet.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/propsh… ============================================================================== --- trunk/rostests/winetests/comctl32/propsheet.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/propsheet.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -750,7 +750,7 @@ static void test_PSM_ADDPAGE(void) { - HPROPSHEETPAGE hpsp[3]; + HPROPSHEETPAGE hpsp[5]; PROPSHEETPAGEA psp; PROPSHEETHEADERA psh; HWND hdlg, tab; @@ -771,6 +771,12 @@ hpsp[1] = CreatePropertySheetPageA(&psp); hpsp[2] = CreatePropertySheetPageA(&psp); + U(psp).pszTemplate = MAKEINTRESOURCE(IDD_PROP_PAGE_ERROR); + hpsp[3] = CreatePropertySheetPageA(&psp); + + psp.dwFlags = PSP_PREMATURE; + hpsp[4] = CreatePropertySheetPageA(&psp); + memset(&psh, 0, sizeof(psh)); psh.dwSize = PROPSHEETHEADERA_V1_SIZE; psh.dwFlags = PSH_MODELESS; @@ -803,6 +809,27 @@ ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)hpsp[2]); ok(ret == TRUE, "got %d\n", ret); + + r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); + ok(r == 3, "got %d\n", r); + + /* add property sheet page that can't be created */ + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)hpsp[3]); + ok(ret == TRUE, "got %d\n", ret); + + r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); + ok(r == 4, "got %d\n", r); + + /* select page that can't be created */ + ret = SendMessageA(hdlg, PSM_SETCURSEL, 3, 0); + ok(ret == TRUE, "got %d\n", ret); + + r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); + ok(r == 3, "got %d\n", r); + + /* test PSP_PREMATURE flag with incorrect property sheet page */ + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)hpsp[4]); + ok(ret == FALSE, "got %d\n", ret); r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); ok(r == 3, "got %d\n", r); Modified: trunk/rostests/winetests/comctl32/resources.h URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/resour… ============================================================================== --- trunk/rostests/winetests/comctl32/resources.h [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/resources.h [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -38,6 +38,7 @@ #define IDD_PROP_PAGE_WITH_CUSTOM_DEFAULT_BUTTON 34 #define IDD_PROP_PAGE_MESSAGE_TEST 35 +#define IDD_PROP_PAGE_ERROR 36 #define IDC_PS_EDIT1 1000 #define IDC_PS_EDIT2 1001 Modified: trunk/rostests/winetests/comctl32/rsrc.rc URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/rsrc.r… ============================================================================== --- trunk/rostests/winetests/comctl32/rsrc.rc [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/rsrc.rc [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -22,7 +22,7 @@ #include "winuser.h" #include "resources.h" -PROP_PAGE1 DIALOG LOADONCALL MOVEABLE DISCARDABLE 5, 43, 227, 215 +PROP_PAGE1 DIALOG 5, 43, 227, 215 STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE CAPTION "Page1" FONT 8, "MS Shell Dlg" @@ -30,7 +30,7 @@ LTEXT "Test", -1, 10, 6, 100, 8 } -IDD_PROP_PAGE_INTRO DIALOG LOADONCALL MOVEABLE DISCARDABLE 5, 43, 227, 215 +IDD_PROP_PAGE_INTRO DIALOG 5, 43, 227, 215 STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE CAPTION "Edit Control Page" FONT 8, "MS Shell Dlg" @@ -38,7 +38,7 @@ LTEXT "This is a test property sheet!", -1, 10, 6, 100, 8 } -IDD_PROP_PAGE_EDIT DIALOG LOADONCALL MOVEABLE DISCARDABLE 5, 43, 227, 215 +IDD_PROP_PAGE_EDIT DIALOG 5, 43, 227, 215 STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE CAPTION "Edit Control Page" FONT 8, "MS Shell Dlg" @@ -47,7 +47,7 @@ EDITTEXT IDC_PS_EDIT2, 5, 160, 150, 28, WS_CHILD | WS_VISIBLE | WS_TABSTOP | ES_MULTILINE } -IDD_PROP_PAGE_RADIO DIALOG LOADONCALL MOVEABLE DISCARDABLE 5, 43, 227, 215 +IDD_PROP_PAGE_RADIO DIALOG 5, 43, 227, 215 STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE CAPTION "Edit Control Page" FONT 8, "MS Shell Dlg" @@ -56,12 +56,26 @@ CONTROL "Radio2", IDC_PS_RADIO2, "Button", BS_AUTORADIOBUTTON, 20, 40, 39, 10 } -IDD_PROP_PAGE_EXIT DIALOG LOADONCALL MOVEABLE DISCARDABLE 5, 43, 227, 215 +IDD_PROP_PAGE_EXIT DIALOG 5, 43, 227, 215 STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE CAPTION "Edit Control Page" FONT 8, "MS Shell Dlg" { LTEXT "This has been a test property sheet!", -1, 10, 6, 170, 8 +} + +IDD_PROP_PAGE_MESSAGE_TEST DIALOG 0, 0, 339, 179 +STYLE DS_MODALFRAME | DS_3DLOOK | DS_CENTER | WS_CHILD | WS_VISIBLE | WS_CAPTION | WS_SYSMENU +CAPTION "Message Test" +BEGIN + LTEXT "Some Text",-1,115,1,195,24 +END + +IDD_PROP_PAGE_ERROR DIALOG 0, 0, 100, 100 +STYLE WS_POPUP | WS_CAPTION | WS_CLIPSIBLINGS | WS_VISIBLE +CLASS "Non-existing class" +FONT 8, "MS Shell Dlg" +{ } STRINGTABLE @@ -80,7 +94,7 @@ /* @makedep: bmp80x15.bmp */ IDB_BITMAP_80x15 BITMAP bmp80x15.bmp -IDD_PROP_PAGE_WITH_CUSTOM_DEFAULT_BUTTON DIALOG DISCARDABLE 5, 43, 227, 215 +IDD_PROP_PAGE_WITH_CUSTOM_DEFAULT_BUTTON DIALOG 5, 43, 227, 215 STYLE WS_CHILD | WS_DISABLED FONT 8, "MS Shell Dlg" { Modified: trunk/rostests/winetests/comctl32/status.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/status… ============================================================================== --- trunk/rostests/winetests/comctl32/status.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/status.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -267,11 +267,11 @@ /* Divide into parts and set text */ r = SendMessage(hWndStatus, SB_SETPARTS, 3, (LPARAM)nParts); expect(TRUE,r); - r = SendMessage(hWndStatus, SB_SETTEXT, 0, (LPARAM)"First"); - expect(TRUE,r); - r = SendMessage(hWndStatus, SB_SETTEXT, 1, (LPARAM)"Second"); - expect(TRUE,r); - r = SendMessage(hWndStatus, SB_SETTEXT, 2, (LPARAM)"Third"); + r = SendMessage(hWndStatus, SB_SETTEXT, SBT_POPOUT|0, (LPARAM)"First"); + expect(TRUE,r); + r = SendMessage(hWndStatus, SB_SETTEXT, SBT_OWNERDRAW|1, (LPARAM)"Second"); + expect(TRUE,r); + r = SendMessage(hWndStatus, SB_SETTEXT, SBT_NOBORDERS|2, (LPARAM)"Third"); expect(TRUE,r); /* Get RECT Information */ @@ -287,13 +287,21 @@ r = SendMessage(hWndStatus, SB_GETRECT, 3, (LPARAM)&rc); expect(FALSE,r); /* Get text length and text */ + r = SendMessage(hWndStatus, SB_GETTEXTLENGTH, 0, 0); + expect(5,LOWORD(r)); + expect(SBT_POPOUT,HIWORD(r)); + r = SendMessageW(hWndStatus, WM_GETTEXTLENGTH, 0, 0); + ok(r == 5 || broken(0x02000005 /* NT4 */), "Expected 5, got %d\n", r); + r = SendMessage(hWndStatus, SB_GETTEXTLENGTH, 1, 0); + expect(0,LOWORD(r)); + expect(SBT_OWNERDRAW,HIWORD(r)); r = SendMessage(hWndStatus, SB_GETTEXTLENGTH, 2, 0); expect(5,LOWORD(r)); - expect(0,HIWORD(r)); + expect(SBT_NOBORDERS,HIWORD(r)); r = SendMessage(hWndStatus, SB_GETTEXT, 2, (LPARAM) charArray); ok(strcmp(charArray,"Third") == 0, "Expected Third, got %s\n", charArray); expect(5,LOWORD(r)); - expect(0,HIWORD(r)); + expect(SBT_NOBORDERS,HIWORD(r)); /* Get parts and borders */ r = SendMessage(hWndStatus, SB_GETPARTS, 3, (LPARAM)checkParts); Added: trunk/rostests/winetests/comctl32/syslink.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/syslin… ============================================================================== --- trunk/rostests/winetests/comctl32/syslink.c (added) +++ trunk/rostests/winetests/comctl32/syslink.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -1,0 +1,257 @@ +/* Unit tests for the syslink control. + * + * Copyright 2011 Francois Gouget for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <windows.h> +#include <commctrl.h> + +#include "wine/test.h" +#include "v6util.h" +#include "msg.h" + +#define expect(expected, got) ok(got == expected, "Expected %d, got %d\n", expected, got) +#define NUM_MSG_SEQUENCE 2 +#define PARENT_SEQ_INDEX 0 +#define SYSLINK_SEQ_INDEX 1 + +static HWND hWndParent; + +static struct msg_sequence *sequences[NUM_MSG_SEQUENCE]; + +static const struct message empty_wnd_seq[] = { + {0} +}; + +static const struct message parent_create_syslink_wnd_seq[] = { + { WM_GETFONT, sent|optional}, /* Only on XP */ + { WM_QUERYUISTATE, sent|optional}, + { WM_CTLCOLORSTATIC, sent}, + { WM_NOTIFY, sent|wparam, 0}, + { WM_PARENTNOTIFY, sent|wparam, WM_CREATE}, + {0} +}; + +static const struct message visible_syslink_wnd_seq[] = { + { WM_STYLECHANGING, sent|wparam, GWL_STYLE}, + { WM_STYLECHANGED, sent|wparam, GWL_STYLE}, + { WM_PAINT, sent}, + {0} +}; + +static const struct message parent_visible_syslink_wnd_seq[] = { + { WM_CTLCOLORSTATIC, sent}, + { WM_NOTIFY, sent|wparam, 0}, + {0} +}; + +/* Try to make sure pending X events have been processed before continuing */ +static void flush_events(void) +{ + MSG msg; + int diff = 200; + int min_timeout = 100; + DWORD time = GetTickCount() + diff; + + while (diff > 0) + { + if (MsgWaitForMultipleObjects( 0, NULL, FALSE, min_timeout, QS_ALLINPUT ) == WAIT_TIMEOUT) break; + while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg ); + diff = time - GetTickCount(); + } +} + +static LRESULT WINAPI parent_wnd_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + static LONG defwndproc_counter = 0; + LRESULT ret; + struct message msg; + + /* log system messages, except for painting */ + if (message < WM_USER && + message != WM_PAINT && + message != WM_ERASEBKGND && + message != WM_NCPAINT && + message != WM_NCHITTEST && + message != WM_GETTEXT && + message != WM_GETICON && + message != WM_DEVICECHANGE) + { + trace("parent: %p, %04x, %08lx, %08lx\n", hwnd, message, wParam, lParam); + + msg.message = message; + msg.flags = sent|wparam|lparam; + if (defwndproc_counter) msg.flags |= defwinproc; + msg.wParam = wParam; + msg.lParam = lParam; + add_message(sequences, PARENT_SEQ_INDEX, &msg); + } + + defwndproc_counter++; + ret = DefWindowProcW(hwnd, message, wParam, lParam); + defwndproc_counter--; + + return ret; +} + +static const WCHAR parentClassW[] = {'S','y','s','l','i','n','k',' ','t','e','s','t',' ','p','a','r','e','n','t',' ','c','l','a','s','s',0}; + +static BOOL register_parent_wnd_class(void) +{ + WNDCLASSW cls; + + cls.style = 0; + cls.lpfnWndProc = parent_wnd_proc; + cls.cbClsExtra = 0; + cls.cbWndExtra = 0; + cls.hInstance = GetModuleHandleW(NULL); + cls.hIcon = 0; + cls.hCursor = LoadCursorW(0, (LPCWSTR)IDC_ARROW); + cls.hbrBackground = GetStockObject(WHITE_BRUSH); + cls.lpszMenuName = NULL; + cls.lpszClassName = parentClassW; + return RegisterClassW(&cls); +} + +static HWND create_parent_window(void) +{ + static const WCHAR titleW[] = {'S','y','s','l','i','n','k',' ','t','e','s','t',' ','p','a','r','e','n','t',' ','w','i','n','d','o','w',0}; + if (!register_parent_wnd_class()) + return NULL; + + return CreateWindowExW(0, parentClassW, titleW, + WS_CAPTION | WS_SYSMENU | WS_MINIMIZEBOX | + WS_MAXIMIZEBOX | WS_VISIBLE, + 0, 0, 200, 100, GetDesktopWindow(), + NULL, GetModuleHandleW(NULL), NULL); +} + +static WNDPROC syslink_oldproc; + +static LRESULT WINAPI syslink_subclass_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + static LONG defwndproc_counter = 0; + LRESULT ret; + struct message msg; + + trace("syslink: %p, %04x, %08lx, %08lx\n", hwnd, message, wParam, lParam); + + msg.message = message; + msg.flags = sent|wparam|lparam; + if (defwndproc_counter) msg.flags |= defwinproc; + msg.wParam = wParam; + msg.lParam = lParam; + add_message(sequences, SYSLINK_SEQ_INDEX, &msg); + + defwndproc_counter++; + ret = CallWindowProcW(syslink_oldproc, hwnd, message, wParam, lParam); + defwndproc_counter--; + + return ret; +} + +static HWND create_syslink(DWORD style, HWND parent) +{ + HWND hWndSysLink; + static const WCHAR linkW[] = {'H','e','a','d',' ','<','a',' ','h','r','e','f','=','"','l','i','n','k','1','"','>','N','a','m','e','1','<','/','a','>',' ','M','i','d','d','l','e',' ','<','a',' ','h','r','e','f','=','"','l','i','n','k','2','"','>','N','a','m','e','2','<','/','a','>',' ','T','a','i','l',0}; + + /* Only Unicode will do here */ + hWndSysLink = CreateWindowExW(0, WC_LINK, linkW, + style, 0, 0, 150, 50, + parent, NULL, GetModuleHandleW(NULL), NULL); + if (!hWndSysLink) return NULL; + + if (GetWindowLongPtrW(hWndSysLink, GWLP_USERDATA)) + /* On Windows XP SysLink takes GWLP_USERDATA for itself! */ + trace("SysLink makes use of GWLP_USERDATA\n"); + + syslink_oldproc = (WNDPROC)SetWindowLongPtrW(hWndSysLink, GWLP_WNDPROC, (LONG_PTR)syslink_subclass_proc); + + return hWndSysLink; +} + + +START_TEST(syslink) +{ + ULONG_PTR ctx_cookie; + HANDLE hCtx; + HMODULE hComctl32; + BOOL (WINAPI *pInitCommonControlsEx)(const INITCOMMONCONTROLSEX*); + INITCOMMONCONTROLSEX iccex; + BOOL rc; + HWND hWndSysLink; + LONG oldstyle; + + if (!load_v6_module(&ctx_cookie, &hCtx)) + return; + + /* LoadLibrary is needed. This file has no reference to functions in comctl32 */ + hComctl32 = LoadLibraryA("comctl32.dll"); + pInitCommonControlsEx = (void*)GetProcAddress(hComctl32, "InitCommonControlsEx"); + if (!pInitCommonControlsEx) + { + win_skip("InitCommonControlsEx() is missing. Skipping the tests\n"); + return; + } + iccex.dwSize = sizeof(iccex); + iccex.dwICC = ICC_LINK_CLASS; + rc = pInitCommonControlsEx(&iccex); + ok(rc, "InitCommonControlsEx failed (le %u)\n", GetLastError()); + if (!rc) + { + skip("Could not register ICC_LINK_CLASS\n"); + return; + } + + init_msg_sequences(sequences, NUM_MSG_SEQUENCE); + + /* Create parent window */ + hWndParent = create_parent_window(); + ok(hWndParent != NULL, "Failed to create parent Window!\n"); + if (!hWndParent) + { + skip("Parent window not present\n"); + return; + } + flush_events(); + + /* Create an invisible SysLink control */ + flush_sequences(sequences, NUM_MSG_SEQUENCE); + hWndSysLink = create_syslink(WS_CHILD | WS_TABSTOP, hWndParent); + ok(hWndSysLink != NULL, "Expected non NULL value (le %u)\n", GetLastError()); + if (!hWndSysLink) + { + skip("SysLink control not present?\n"); + return; + } + flush_events(); + ok_sequence(sequences, SYSLINK_SEQ_INDEX, empty_wnd_seq, "create SysLink", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_create_syslink_wnd_seq, "create SysLink (parent)", TRUE); + + /* Make the SysLink control visible */ + flush_sequences(sequences, NUM_MSG_SEQUENCE); + oldstyle = GetWindowLong(hWndSysLink, GWL_STYLE); + SetWindowLong(hWndSysLink, GWL_STYLE, oldstyle | WS_VISIBLE); + RedrawWindow(hWndSysLink, NULL, NULL, RDW_INVALIDATE); + flush_events(); + ok_sequence(sequences, SYSLINK_SEQ_INDEX, visible_syslink_wnd_seq, "visible SysLink", TRUE); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_visible_syslink_wnd_seq, "visible SysLink (parent)", TRUE); + + DestroyWindow(hWndSysLink); + DestroyWindow(hWndParent); + unload_v6_module(ctx_cookie, hCtx); +} Propchange: trunk/rostests/winetests/comctl32/syslink.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/rostests/winetests/comctl32/tab.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/tab.c?… ============================================================================== --- trunk/rostests/winetests/comctl32/tab.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/tab.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -47,15 +47,11 @@ SendMessage(hwnd, TCM_GETITEMRECT, 0, (LPARAM)&r); if (width >= 0 && height < 0) - { - ok_(__FILE__,line) (width == r.right - r.left, "%s: Expected width [%d] got [%d]\n",\ - msg, width, r.right - r.left); - } + ok_(__FILE__,line) (width == r.right - r.left, "%s: Expected width [%d] got [%d]\n", + msg, width, r.right - r.left); else if (height >= 0 && width < 0) - { - ok_(__FILE__,line) (height == r.bottom - r.top, "%s: Expected height [%d] got [%d]\n",\ - msg, height, r.bottom - r.top); - } + ok_(__FILE__,line) (height == r.bottom - r.top, "%s: Expected height [%d] got [%d]\n", + msg, height, r.bottom - r.top); else ok_(__FILE__,line) ((width == r.right - r.left) && (height == r.bottom - r.top ), "%s: Expected [%d,%d] got [%d,%d]\n", msg, width, height, @@ -661,7 +657,6 @@ DestroyWindow (hwTab); ImageList_Destroy(himl); - DeleteObject(hFont); } static void test_width(void) @@ -1428,6 +1423,36 @@ DestroyWindow(hTab); } +struct tabcreate_style { + DWORD style; + DWORD act_style; +}; + +static const struct tabcreate_style create_styles[] = +{ + { WS_CHILD|TCS_BOTTOM|TCS_VERTICAL, WS_CHILD|WS_CLIPSIBLINGS|TCS_BOTTOM|TCS_VERTICAL|TCS_MULTILINE }, + { WS_CHILD|TCS_VERTICAL, WS_CHILD|WS_CLIPSIBLINGS|TCS_VERTICAL|TCS_MULTILINE }, + { 0 } +}; + +static void test_create(void) +{ + const struct tabcreate_style *ptr = create_styles; + DWORD style; + HWND hTab; + + while (ptr->style) + { + hTab = CreateWindowA(WC_TABCONTROLA, "TestTab", ptr->style, + 10, 10, 300, 100, parent_wnd, NULL, NULL, 0); + style = GetWindowLongA(hTab, GWL_STYLE); + ok(style == ptr->act_style, "expected style 0x%08x, got style 0x%08x\n", ptr->act_style, style); + + DestroyWindow(hTab); + ptr++; + } +} + START_TEST(tab) { LOGFONTA logfont; @@ -1465,6 +1490,7 @@ test_TCM_SETITEMEXTRA(); test_TCS_OWNERDRAWFIXED(); test_WM_CONTEXTMENU(); + test_create(); DestroyWindow(parent_wnd); } Modified: trunk/rostests/winetests/comctl32/testlist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/testli… ============================================================================== --- trunk/rostests/winetests/comctl32/testlist.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/testlist.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -11,22 +11,24 @@ extern void func_dpa(void); extern void func_header(void); extern void func_imagelist(void); +extern void func_ipaddress(void); extern void func_listview(void); extern void func_misc(void); extern void func_monthcal(void); extern void func_mru(void); +extern void func_pager(void); extern void func_progress(void); extern void func_propsheet(void); extern void func_rebar(void); extern void func_status(void); extern void func_subclass(void); +extern void func_syslink(void); extern void func_tab(void); extern void func_toolbar(void); extern void func_tooltips(void); extern void func_trackbar(void); extern void func_treeview(void); extern void func_updown(void); -extern void func_ipaddress(void); const struct test winetest_testlist[] = { @@ -35,21 +37,23 @@ { "dpa", func_dpa }, { "header", func_header }, { "imagelist", func_imagelist }, + { "ipaddress", func_ipaddress }, { "listview", func_listview }, { "misc", func_misc }, { "monthcal", func_monthcal }, { "mru", func_mru }, + { "pager", func_pager }, { "progress", func_progress }, { "propsheet", func_propsheet }, { "rebar", func_rebar }, { "status", func_status }, { "subclass", func_subclass }, + { "syslink", func_syslink }, { "tab", func_tab }, { "toolbar", func_toolbar }, { "tooltips", func_tooltips }, { "trackbar", func_trackbar }, { "treeview", func_treeview }, { "updown", func_updown }, - { "ipaddress", func_ipaddress }, { 0, 0 } }; Modified: trunk/rostests/winetests/comctl32/toolbar.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/toolba… ============================================================================== --- trunk/rostests/winetests/comctl32/toolbar.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/toolbar.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -53,6 +53,24 @@ { WM_NOTIFY, sent|id, 0, 0, TTN_GETDISPINFOA }, { 0 } }; + +#define DEFINE_EXPECT(func) \ + static BOOL expect_ ## func = FALSE, called_ ## func = FALSE + +#define CHECK_EXPECT2(func) \ + do { \ + ok(expect_ ##func, "unexpected call " #func "\n"); \ + called_ ## func = TRUE; \ + }while(0) + +#define CHECK_CALLED(func) \ + do { \ + ok(called_ ## func, "expected " #func "\n"); \ + expect_ ## func = called_ ## func = FALSE; \ + }while(0) + +#define SET_EXPECT(func) \ + expect_ ## func = TRUE #define expect(EXPECTED,GOT) ok((GOT)==(EXPECTED), "Expected %d, got %d\n", (EXPECTED), (GOT)) @@ -1610,6 +1628,7 @@ {0, 21, TBSTATE_ENABLED, 0, {0, }, 0, -1}, }; NMTTDISPINFOW nmtti; + HWND tooltip; rebuild_toolbar(&hToolbar); @@ -1631,6 +1650,13 @@ SendMessageA(hToolbar, WM_NOTIFY, 0, (LPARAM)&nmtti); g_ResetDispTextPtr = FALSE; + DestroyWindow(hToolbar); + + /* TBSTYLE_TOOLTIPS */ + hToolbar = CreateWindowExA(0, TOOLBARCLASSNAME, NULL, WS_CHILD | WS_VISIBLE, 0, 0, 0, 0, + hMainWnd, (HMENU)5, GetModuleHandle(NULL), NULL); + tooltip = (HWND)SendMessageA(hToolbar, TB_GETTOOLTIPS, 0, 0); + ok(tooltip == NULL, "got %p\n", tooltip); DestroyWindow(hToolbar); } @@ -1658,6 +1684,7 @@ style = SendMessageA(hToolbar, TB_GETSTYLE, 0, 0); style2 = GetWindowLongA(hToolbar, GWL_STYLE); +todo_wine ok(style == style2, "got 0x%08x, expected 0x%08x\n", style, style2); /* try to alter common window bits */ @@ -1679,6 +1706,208 @@ ok(style == style2, "got 0x%08x, expected 0x%08x\n", style, style2); DestroyWindow(hToolbar); +} + +static HHOOK g_tbhook; +static HWND g_toolbar; + +DEFINE_EXPECT(g_hook_create); +DEFINE_EXPECT(g_hook_WM_NCCREATE); +DEFINE_EXPECT(g_hook_WM_CREATE); + +static LRESULT WINAPI toolbar_subclass_proc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam) +{ + WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); + LRESULT ret; + DWORD style; + + if (msg == WM_NCCREATE) + { + if (g_toolbar == hwnd) + { + CHECK_EXPECT2(g_hook_WM_NCCREATE); + g_toolbar = hwnd; + ret = CallWindowProcA(oldproc, hwnd, msg, wParam, lParam); + + /* control is already set up */ + style = SendMessageA(hwnd, TB_GETSTYLE, 0, 0); + ok(style != 0, "got %x\n", style); + + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TOOLTIPS) == 0, "got 0x%08x\n", style); + SetWindowLongA(hwnd, GWL_STYLE, style|TBSTYLE_TOOLTIPS); + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%08x\n", style); + + return ret; + } + } + else if (msg == WM_CREATE) + { + CREATESTRUCTA *cs = (CREATESTRUCTA*)lParam; + + if (g_toolbar == hwnd) + { + CHECK_EXPECT2(g_hook_WM_CREATE); + + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%08x\n", style); + + /* test if toolbar-specific messages are already working before WM_CREATE */ + style = SendMessageA(hwnd, TB_GETSTYLE, 0, 0); + ok(style != 0, "got %x\n", style); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%x\n", style); + ok((cs->style & TBSTYLE_TOOLTIPS) == 0, "0x%08x\n", cs->style); + + ret = CallWindowProcA(oldproc, hwnd, msg, wParam, lParam); + + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%08x\n", style); + + /* test if toolbar-specific messages are already working before WM_CREATE */ + style = SendMessageA(hwnd, TB_GETSTYLE, 0, 0); + ok(style != 0, "got %x\n", style); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%x\n", style); + + return ret; + } + } + + return CallWindowProcA(oldproc, hwnd, msg, wParam, lParam); +} + +static LRESULT CALLBACK cbt_hook_proc(int code, WPARAM wParam, LPARAM lParam) +{ + if (code == HCBT_CREATEWND) + { + HWND hwnd = (HWND)wParam; + + if (!g_toolbar) + { + WNDPROC oldproc; + + CHECK_EXPECT2(g_hook_create); + g_toolbar = hwnd; + /* subclass */ + oldproc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, (LONG_PTR)toolbar_subclass_proc); + SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); + } + return 0; + } + + return CallNextHookEx(g_tbhook, code, wParam, lParam); +} + +static void test_create(void) +{ + HWND hwnd, tooltip; + DWORD style; + + g_tbhook = SetWindowsHookA(WH_CBT, cbt_hook_proc); + + SET_EXPECT(g_hook_create); + SET_EXPECT(g_hook_WM_NCCREATE); + SET_EXPECT(g_hook_WM_CREATE); + + hwnd = CreateWindowExA(0, TOOLBARCLASSNAME, NULL, WS_CHILD | WS_VISIBLE, 0, 0, 0, 0, + hMainWnd, (HMENU)5, GetModuleHandle(NULL), NULL); + + CHECK_CALLED(g_hook_create); + CHECK_CALLED(g_hook_WM_NCCREATE); + CHECK_CALLED(g_hook_WM_CREATE); + + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TOOLTIPS) == TBSTYLE_TOOLTIPS, "got 0x%08x\n", style); + + tooltip = (HWND)SendMessageA(hwnd, TB_GETTOOLTIPS, 0, 0); + ok(tooltip != NULL, "got %p\n", tooltip); + ok(GetParent(tooltip) == hMainWnd, "got %p, %p\n", hMainWnd, hwnd); + + DestroyWindow(hwnd); + UnhookWindowsHook(WH_CBT, cbt_hook_proc); + + /* TBSTYLE_TRANSPARENT */ + hwnd = CreateWindowExA(0, TOOLBARCLASSNAME, NULL, + WS_CHILD|WS_VISIBLE|WS_CLIPSIBLINGS|TBSTYLE_FLAT|TBSTYLE_TOOLTIPS|TBSTYLE_GROUP, + 0, 0, 0, 0, hMainWnd, (HMENU)5, GetModuleHandle(NULL), NULL); + + style = GetWindowLongA(hwnd, GWL_STYLE); + ok((style & TBSTYLE_TRANSPARENT) == TBSTYLE_TRANSPARENT, "got 0x%08x\n", style); + + style = SendMessageA(hwnd, TB_GETSTYLE, 0, 0); + ok((style & TBSTYLE_TRANSPARENT) == TBSTYLE_TRANSPARENT, "got 0x%08x\n", style); + + DestroyWindow(hwnd); +} + +typedef struct { + DWORD mask; + DWORD style; + DWORD style_set; +} extended_style_t; + +static const extended_style_t extended_style_test[] = { + { + TBSTYLE_EX_DRAWDDARROWS | TBSTYLE_EX_HIDECLIPPEDBUTTONS | TBSTYLE_EX_DOUBLEBUFFER, + TBSTYLE_EX_DRAWDDARROWS | TBSTYLE_EX_HIDECLIPPEDBUTTONS | TBSTYLE_EX_DOUBLEBUFFER, + TBSTYLE_EX_DRAWDDARROWS | TBSTYLE_EX_HIDECLIPPEDBUTTONS | TBSTYLE_EX_DOUBLEBUFFER + }, + { + TBSTYLE_EX_MIXEDBUTTONS, TBSTYLE_EX_MIXEDBUTTONS, + TBSTYLE_EX_DRAWDDARROWS | TBSTYLE_EX_HIDECLIPPEDBUTTONS | TBSTYLE_EX_DOUBLEBUFFER | TBSTYLE_EX_MIXEDBUTTONS + }, + + { 0, TBSTYLE_EX_MIXEDBUTTONS, TBSTYLE_EX_MIXEDBUTTONS }, + { 0, 0, 0 }, + { 0, TBSTYLE_EX_DRAWDDARROWS, TBSTYLE_EX_DRAWDDARROWS }, + { 0, TBSTYLE_EX_HIDECLIPPEDBUTTONS, TBSTYLE_EX_HIDECLIPPEDBUTTONS }, + + { 0, 0, 0 }, + { TBSTYLE_EX_HIDECLIPPEDBUTTONS, TBSTYLE_EX_MIXEDBUTTONS, 0 }, + { TBSTYLE_EX_MIXEDBUTTONS, TBSTYLE_EX_HIDECLIPPEDBUTTONS, 0 }, + { TBSTYLE_EX_DOUBLEBUFFER, TBSTYLE_EX_MIXEDBUTTONS, 0 }, + + { + TBSTYLE_EX_DOUBLEBUFFER | TBSTYLE_EX_MIXEDBUTTONS, + TBSTYLE_EX_MIXEDBUTTONS, TBSTYLE_EX_MIXEDBUTTONS + }, + { + TBSTYLE_EX_DOUBLEBUFFER | TBSTYLE_EX_MIXEDBUTTONS, + TBSTYLE_EX_DOUBLEBUFFER, TBSTYLE_EX_DOUBLEBUFFER + } +}; + +static void test_TB_GET_SET_EXTENDEDSTYLE(void) +{ + DWORD style, oldstyle, oldstyle2; + const extended_style_t *ptr; + HWND hwnd = NULL; + int i; + + rebuild_toolbar(&hwnd); + + SendMessageA(hwnd, TB_SETEXTENDEDSTYLE, TBSTYLE_EX_DOUBLEBUFFER, TBSTYLE_EX_MIXEDBUTTONS); + style = SendMessageA(hwnd, TB_GETEXTENDEDSTYLE, 0, 0); + if (style == TBSTYLE_EX_MIXEDBUTTONS) + { + win_skip("Some extended style bits are not supported\n"); + DestroyWindow(hwnd); + return; + } + + for (i = 0; i < sizeof(extended_style_test)/sizeof(extended_style_t); i++) + { + ptr = &extended_style_test[i]; + + oldstyle2 = SendMessageA(hwnd, TB_GETEXTENDEDSTYLE, 0, 0); + + oldstyle = SendMessageA(hwnd, TB_SETEXTENDEDSTYLE, ptr->mask, ptr->style); + ok(oldstyle == oldstyle2, "%d: got old style 0x%08x, expected 0x%08x\n", i, oldstyle, oldstyle2); + style = SendMessageA(hwnd, TB_GETEXTENDEDSTYLE, 0, 0); + ok(style == ptr->style_set, "%d: got style 0x%08x, expected 0x%08x\n", i, style, ptr->style_set); + } + + DestroyWindow(hwnd); } START_TEST(toolbar) @@ -1721,6 +1950,8 @@ test_getstring(); test_tooltip(); test_get_set_style(); + test_create(); + test_TB_GET_SET_EXTENDEDSTYLE(); PostQuitMessage(0); while(GetMessageA(&msg,0,0,0)) { Modified: trunk/rostests/winetests/comctl32/tooltips.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/toolti… ============================================================================== --- trunk/rostests/winetests/comctl32/tooltips.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/tooltips.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -322,6 +322,11 @@ toolinfoA.lpszText = bufA; SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, "") == 0, "lpszText should be an empty string\n"); + + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(toolinfoA.lpszText == NULL, + "expected NULL, got %p\n", toolinfoA.lpszText); } else { @@ -355,6 +360,12 @@ SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, testtipA) == 0, "lpszText should be an empty string\n"); + memset(bufA, 0x1f, sizeof(bufA)); + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(strcmp(toolinfoA.lpszText, testtipA) == 0, + "expected %s, got %p\n", testtipA, toolinfoA.lpszText); + length = SendMessage(hwnd, WM_GETTEXTLENGTH, 0, 0); ok(length == 0, "Expected 0, got %d\n", length); } @@ -378,6 +389,11 @@ SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, testcallbackA) == 0, "lpszText should be an (%s) string\n", testcallbackA); + + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(toolinfoA.lpszText == LPSTR_TEXTCALLBACKA, + "expected LPSTR_TEXTCALLBACKA, got %p\n", toolinfoA.lpszText); } DestroyWindow(hwnd); @@ -620,6 +636,128 @@ DestroyWindow(hwnd); } +static void test_longtextA(void) +{ + static const char longtextA[] = + "According to MSDN, TTM_ENUMTOOLS claims that TOOLINFO's lpszText is maximum " + "80 chars including null. In fact, the buffer is not null-terminated."; + HWND hwnd; + TTTOOLINFOA toolinfoA = { 0 }; + CHAR bufA[256]; + LRESULT r; + + hwnd = CreateWindowExA(0, TOOLTIPS_CLASSA, NULL, 0, + 10, 10, 300, 100, + NULL, NULL, NULL, 0); + toolinfoA.cbSize = sizeof(TTTOOLINFOA); + toolinfoA.hwnd = NULL; + toolinfoA.hinst = GetModuleHandleA(NULL); + toolinfoA.uFlags = 0; + toolinfoA.uId = 0x1234ABCD; + strcpy(bufA, longtextA); + toolinfoA.lpszText = bufA; + toolinfoA.lParam = 0xdeadbeef; + GetClientRect(hwnd, &toolinfoA.rect); + r = SendMessageA(hwnd, TTM_ADDTOOLA, 0, (LPARAM)&toolinfoA); + if (r) + { + int textlen; + memset(bufA, 0, sizeof(bufA)); + toolinfoA.hwnd = NULL; + toolinfoA.uId = 0xABCD1234; + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_ENUMTOOLSA, 0, (LPARAM)&toolinfoA); + textlen = lstrlenA(toolinfoA.lpszText); + ok(textlen == 80, "lpszText has %d chars\n", textlen); + ok(toolinfoA.uId == 0x1234ABCD, + "uId should be retrieved, got %p\n", (void*)toolinfoA.uId); + + memset(bufA, 0, sizeof(bufA)); + toolinfoA.hwnd = NULL; + toolinfoA.uId = 0x1234ABCD; + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + textlen = lstrlenA(toolinfoA.lpszText); + ok(textlen == 80, "lpszText has %d chars\n", textlen); + + memset(bufA, 0, sizeof(bufA)); + toolinfoA.hwnd = NULL; + toolinfoA.uId = 0x1234ABCD; + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); + textlen = lstrlenA(toolinfoA.lpszText); + ok(textlen == 80, "lpszText has %d chars\n", textlen); + } + + DestroyWindow(hwnd); +} + +static void test_longtextW(void) +{ + static const char longtextA[] = + "According to MSDN, TTM_ENUMTOOLS claims that TOOLINFO's lpszText is maximum " + "80 chars including null. Actually, this is not applied for wide version."; + HWND hwnd; + TTTOOLINFOW toolinfoW = { 0 }; + WCHAR bufW[256]; + LRESULT r; + int lenW; + + hwnd = CreateWindowExW(0, TOOLTIPS_CLASSW, NULL, 0, + 10, 10, 300, 100, + NULL, NULL, NULL, 0); + if(!hwnd) + { + win_skip("CreateWindowExW() not supported. Skipping.\n"); + return; + } + + toolinfoW.cbSize = TTTOOLINFOW_V2_SIZE; + toolinfoW.hwnd = NULL; + toolinfoW.hinst = GetModuleHandleW(NULL); + toolinfoW.uFlags = 0; + toolinfoW.uId = 0x1234ABCD; + MultiByteToWideChar(CP_ACP, 0, longtextA, -1, bufW, sizeof(bufW)/sizeof(bufW[0])); + lenW = lstrlenW(bufW); + toolinfoW.lpszText = bufW; + toolinfoW.lParam = 0xdeadbeef; + GetClientRect(hwnd, &toolinfoW.rect); + r = SendMessageW(hwnd, TTM_ADDTOOLW, 0, (LPARAM)&toolinfoW); + if (r) + { + int textlen; + memset(bufW, 0, sizeof(bufW)); + toolinfoW.hwnd = NULL; + toolinfoW.uId = 0xABCD1234; + toolinfoW.lpszText = bufW; + SendMessageW(hwnd, TTM_ENUMTOOLSW, 0, (LPARAM)&toolinfoW); + textlen = lstrlenW(toolinfoW.lpszText); + ok(textlen == lenW, "lpszText has %d chars\n", textlen); + ok(toolinfoW.uId == 0x1234ABCD, + "uId should be retrieved, got %p\n", (void*)toolinfoW.uId); + + memset(bufW, 0, sizeof(bufW)); + toolinfoW.hwnd = NULL; + toolinfoW.uId = 0x1234ABCD; + toolinfoW.lpszText = bufW; + SendMessageW(hwnd, TTM_GETTOOLINFOW, 0, (LPARAM)&toolinfoW); + textlen = lstrlenW(toolinfoW.lpszText); + ok(textlen == lenW, "lpszText has %d chars\n", textlen); + + memset(bufW, 0, sizeof(bufW)); + toolinfoW.hwnd = NULL; + toolinfoW.uId = 0x1234ABCD; + toolinfoW.lpszText = bufW; + SendMessageW(hwnd, TTM_GETTEXTW, 0, (LPARAM)&toolinfoW); + textlen = lstrlenW(toolinfoW.lpszText); + ok(textlen == lenW || + broken(textlen == 0 && toolinfoW.lpszText == NULL), /* nt4, kb186177 */ + "lpszText has %d chars\n", textlen); + } + + DestroyWindow(hwnd); +} + START_TEST(tooltips) { InitCommonControls(); @@ -628,4 +766,6 @@ test_customdraw(); test_gettext(); test_ttm_gettoolinfo(); -} + test_longtextA(); + test_longtextW(); +} Modified: trunk/rostests/winetests/comctl32/trackbar.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/trackb… ============================================================================== --- trunk/rostests/winetests/comctl32/trackbar.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/trackbar.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -33,11 +33,7 @@ static struct msg_sequence *sequences[NUM_MSG_SEQUENCE]; -static const struct message create_trackbar_wnd_seq[] = { - {0} -}; - -static const struct message parent_empty_test_seq[] = { +static const struct message empty_seq[] = { {0} }; @@ -298,10 +294,20 @@ { WM_PAINT, sent|defwinproc}, { TBM_GETTHUMBLENGTH, sent}, { TBM_GETTHUMBLENGTH, sent}, + { WM_SIZE, sent}, + { WM_PAINT, sent|defwinproc}, + { TBM_GETTHUMBLENGTH, sent}, + { WM_SIZE, sent}, + { WM_PAINT, sent|defwinproc}, + { TBM_GETTHUMBLENGTH, sent}, {0} }; static const struct message parent_thumb_length_test_seq[] = { + { WM_CTLCOLORSTATIC, sent}, + { WM_NOTIFY, sent}, + { WM_CTLCOLORSTATIC, sent}, + { WM_NOTIFY, sent}, { WM_CTLCOLORSTATIC, sent}, { WM_NOTIFY, sent}, { WM_CTLCOLORSTATIC, sent}, @@ -535,7 +541,7 @@ expect(4, r); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, line_size_test_seq, "linesize test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent line test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent line test sequence", FALSE); } @@ -555,7 +561,7 @@ expect(20, r); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, page_size_test_seq, "page size test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent page size test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent page size test sequence", FALSE); /* check for zero page size */ r = SendMessage(hWndTrackbar, TBM_SETPAGESIZE, 0, 0); @@ -742,8 +748,17 @@ r = SendMessage(hWndTrackbar, TBM_GETTHUMBLENGTH, 0,0); expect(20, r); + r = SendMessage(hWndTrackbar, WM_SIZE, 0,0); + expect(0, r); + r = SendMessage(hWndTrackbar, TBM_GETTHUMBLENGTH, 0,0); + expect(20, r); + r = SendMessage(hWndTrackbar, WM_SIZE, 0, MAKELPARAM(50, 50) ); + expect(0, r); + r = SendMessage(hWndTrackbar, TBM_GETTHUMBLENGTH, 0,0); + expect(20, r); + ok_sequence(sequences, TRACKBAR_SEQ_INDEX, thumb_length_test_seq, "thumb length test sequence", TRUE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_thumb_length_test_seq, "parent thumb lenth test sequence", TRUE); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_thumb_length_test_seq, "parent thumb length test sequence", TRUE); } static void test_tic_settings(HWND hWndTrackbar){ @@ -842,7 +857,7 @@ ok(r > 0, "Expected r > 0, got %d\n", r); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, tic_placement_test_seq, "get tic placement test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent get tic placement test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent get tic placement test sequence", FALSE); } @@ -886,7 +901,7 @@ ok(rTest == hWndTooltip, "Expected hWndTooltip\n"); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, tool_tips_test_seq, "tool tips test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent tool tips test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent tool tips test sequence", FALSE); } @@ -905,7 +920,7 @@ ok(r == FALSE, "Expected FALSE, got %d\n",r); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, unicode_test_seq, "unicode test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent unicode test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent unicode test sequence", FALSE); } static void test_ignore_selection(HWND hWndTrackbar){ @@ -947,7 +962,7 @@ expect(0, r); ok_sequence(sequences, TRACKBAR_SEQ_INDEX, ignore_selection_test_seq, "ignore selection setting test sequence", FALSE); - ok_sequence(sequences, PARENT_SEQ_INDEX, parent_empty_test_seq, "parent ignore selection setting test sequence", FALSE); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "parent ignore selection setting test sequence", FALSE); } static void test_initial_state(void) @@ -963,6 +978,66 @@ expect(-1, ret); ret = SendMessage(hWnd, TBM_GETTICPOS, 0, 0); expect(-1, ret); + ret = SendMessage(hWnd, TBM_GETRANGEMIN, 0, 0); + expect(0, ret); + ret = SendMessage(hWnd, TBM_GETRANGEMAX, 0, 0); + expect(100, ret); + + ret = SendMessage(hWnd, TBM_SETRANGEMAX, TRUE, 200); + expect(0, ret); + + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(2, ret); + + ret = SendMessage(hWnd, TBM_SETRANGEMIN, TRUE, 10); + expect(0, ret); + + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(2, ret); + + DestroyWindow(hWnd); +} + +static void test_TBS_AUTOTICKS(void) +{ + HWND hWnd; + int ret; + + hWnd = create_trackbar(TBS_AUTOTICKS, hWndParent); + + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(2, ret); + ret = SendMessage(hWnd, TBM_GETTIC, 0, 0); + expect(-1, ret); + ret = SendMessage(hWnd, TBM_GETTICPOS, 0, 0); + expect(-1, ret); + ret = SendMessage(hWnd, TBM_GETRANGEMIN, 0, 0); + expect(0, ret); + ret = SendMessage(hWnd, TBM_GETRANGEMAX, 0, 0); + expect(100, ret); + + /* TBM_SETRANGEMAX rebuilds tics */ + ret = SendMessage(hWnd, TBM_SETRANGEMAX, TRUE, 200); + expect(0, ret); + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(201, ret); + + /* TBM_SETRANGEMIN rebuilds tics */ + ret = SendMessage(hWnd, TBM_SETRANGEMAX, TRUE, 100); + expect(0, ret); + ret = SendMessage(hWnd, TBM_SETRANGEMIN, TRUE, 10); + expect(0, ret); + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(91, ret); + + ret = SendMessage(hWnd, TBM_SETRANGEMIN, TRUE, 0); + expect(0, ret); + + /* TBM_SETRANGE rebuilds tics */ + ret = SendMessage(hWnd, TBM_SETRANGE, TRUE, MAKELONG(10, 200)); + expect(0, ret); + ret = SendMessage(hWnd, TBM_GETNUMTICS, 0, 0); + expect(191, ret); DestroyWindow(hWnd); } @@ -996,7 +1071,7 @@ return; } - ok_sequence(sequences, TRACKBAR_SEQ_INDEX, create_trackbar_wnd_seq, "create Trackbar Window", FALSE); + ok_sequence(sequences, TRACKBAR_SEQ_INDEX, empty_seq, "create Trackbar Window", FALSE); ok_sequence(sequences, PARENT_SEQ_INDEX, parent_create_trackbar_wnd_seq, "parent trackbar window", TRUE); flush_sequences(sequences, NUM_MSG_SEQUENCE); @@ -1012,6 +1087,7 @@ test_tic_placement(hWndTrackbar); test_tool_tips(hWndTrackbar); test_unicode(hWndTrackbar); + test_TBS_AUTOTICKS(); flush_sequences(sequences, NUM_MSG_SEQUENCE); DestroyWindow(hWndTrackbar); Modified: trunk/rostests/winetests/comctl32/treeview.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/comctl32/treevi… ============================================================================== --- trunk/rostests/winetests/comctl32/treeview.c [iso-8859-1] (original) +++ trunk/rostests/winetests/comctl32/treeview.c [iso-8859-1] Fri Dec 14 23:24:10 2012 @@ -32,10 +32,6 @@ #include "wine/test.h" #include "v6util.h" #include "msg.h" - -// Hack wine! Not in Sdk CommCtrl.h! -#define TVIS_FOCUSED 0x0001 -// static const char *TEST_CALLBACK_TEXT = "callback_text"; @@ -208,6 +204,27 @@ { 0 } }; +static const struct message parent_expand_kb_seq[] = { + { WM_NOTIFY, sent|id|optional, 0, 0, TVN_KEYDOWN }, + { WM_NOTIFY, sent|id, 0, 0, TVN_ITEMEXPANDINGA }, + { WM_NOTIFY, sent|id, 0, 0, TVN_ITEMEXPANDEDA }, + { WM_CHANGEUISTATE, sent|optional }, + { 0 } +}; + +static const struct message parent_collapse_2nd_kb_seq[] = { + { WM_NOTIFY, sent|id|optional, 0, 0, TVN_KEYDOWN }, + { WM_NOTIFY, sent|id, 0, 0, TVN_ITEMEXPANDINGA }, + { WM_CHANGEUISTATE, sent|optional }, + { 0 } +}; + +static const struct message parent_expand_empty_kb_seq[] = { + { WM_NOTIFY, sent|id|optional, 0, 0, TVN_KEYDOWN }, + { WM_CHANGEUISTATE, sent|optional }, + { 0 } +}; + static const struct message parent_singleexpand_seq[] = { { WM_NOTIFY, sent|id, 0, 0, TVN_SELCHANGINGA }, { WM_NOTIFY, sent|id, 0, 0, TVN_SELCHANGEDA }, @@ -736,8 +753,8 @@ TVITEMA tviRoot = {0}; int nBufferSize = 80; char szBuffer[80] = {0}; + HWND hTree, hTree2; DWORD ret; - HWND hTree; hTree = create_treeview_control(0); fill_tree(hTree); @@ -781,7 +798,18 @@ ok_sequence(sequences, TREEVIEW_SEQ_INDEX, test_get_set_item_seq, "test get set item", FALSE); - DestroyWindow(hTree); + /* get item from a different tree */ + hTree2 = create_treeview_control(0); + + tviRoot.hItem = hRoot; + tviRoot.mask = TVIF_STATE; + tviRoot.state = 0; + ret = SendMessage( hTree2, TVM_GETITEMA, 0, (LPARAM)&tviRoot ); + expect(TRUE, ret); + ok(tviRoot.state == TVIS_FOCUSED, "got state 0x%0x\n", tviRoot.state); + + DestroyWindow(hTree); + DestroyWindow(hTree2); } static void test_get_set_itemheight(void) @@ -934,6 +962,10 @@ hTree = create_treeview_control(0); fill_tree(hTree); + + /* Check that an invalid format returned by NF_QUERY defaults to ANSI */ + bPreviousSetting = (BOOL)SendMessage( hTree, TVM_GETUNICODEFORMAT, 0, 0 ); + ok(bPreviousSetting == 0, "Format should be ANSI.\n"); flush_sequences(sequences, NUM_MSG_SEQUENCES); @@ -986,10 +1018,17 @@ } switch(message) { + case WM_NOTIFYFORMAT: + { + /* Make NF_QUERY return an invalid format to show that it defaults to ANSI */ + if (lParam == NF_QUERY) return 0; + break; + } + case WM_NOTIFY: { NMHDR *pHdr = (NMHDR *)lParam; - + ok(pHdr->code != NM_TOOLTIPSCREATED, "Treeview should not send NM_TOOLTIPSCREATED\n"); if (pHdr->idFrom == 100) { @@ -1050,8 +1089,6 @@ ok(pTreeView->itemNew.mask == (TVIF_HANDLE | TVIF_SELECTEDIMAGE | TVIF_IMAGE | TVIF_PARAM | TVIF_STATE), "got wrong mask %x\n", pTreeView->itemNew.mask); - ok((pTreeView->itemNew.state & TVIS_EXPANDED) == 0, - "got wrong state %x\n", pTreeView->itemNew.state); ok(pTreeView->itemOld.mask == 0, "got wrong mask %x\n", pTreeView->itemOld.mask); @@ -1361,6 +1398,11 @@ ret = SendMessageA(hTree, TVM_SELECTITEM, TVGN_CARET, (LPARAM)hRoot); expect(TRUE, ret); + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, TVM_EXPAND, TVE_COLLAPSE, (LPARAM)hRoot); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "no collapse notifications", FALSE); + g_get_from_expand = TRUE; /* expand */ flush_sequences(sequences, NUM_MSG_SEQUENCES); @@ -1425,6 +1467,67 @@ ret = SendMessageA(hTree, TVM_EXPAND, TVE_TOGGLE, (LPARAM)hRoot); expect(TRUE, ret); ok_sequence(sequences, PARENT_SEQ_INDEX, empty_seq, "toggle node (collapse)", FALSE); + + DestroyWindow(hTree); + + /* some keyboard events are also translated to expand */ + hTree = create_treeview_control(0); + fill_tree(hTree); + + /* preselect root node here */ + ret = SendMessageA(hTree, TVM_SELECTITEM, TVGN_CARET, (LPARAM)hRoot); + expect(TRUE, ret); + + g_get_from_expand = TRUE; + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_ADD, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_kb_seq, "expand node", FALSE); + ok(g_item_expanding.state == TVIS_SELECTED, "got state on TVN_ITEMEXPANDING 0x%08x\n", + g_item_expanding.state); + ok(g_item_expanded.state == (TVIS_SELECTED|TVIS_EXPANDED), "got state on TVN_ITEMEXPANDED 0x%08x\n", + g_item_expanded.state); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_ADD, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_kb_seq, "expand node again", FALSE); + ok(g_item_expanding.state == (TVIS_SELECTED|TVIS_EXPANDED|TVIS_EXPANDEDONCE), "got state on TVN_ITEMEXPANDING 0x%08x\n", + g_item_expanding.state); + ok(g_item_expanded.state == (TVIS_SELECTED|TVIS_EXPANDED|TVIS_EXPANDEDONCE), "got state on TVN_ITEMEXPANDED 0x%08x\n", + g_item_expanded.state); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_SUBTRACT, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_kb_seq, "collapse node", FALSE); + ok(g_item_expanding.state == (TVIS_SELECTED|TVIS_EXPANDED|TVIS_EXPANDEDONCE), "got state on TVN_ITEMEXPANDING 0x%08x\n", + g_item_expanding.state); + ok(g_item_expanded.state == (TVIS_SELECTED|TVIS_EXPANDEDONCE), "got state on TVN_ITEMEXPANDED 0x%08x\n", + g_item_expanded.state); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_SUBTRACT, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_collapse_2nd_kb_seq, "collapse node again", FALSE); + ok(g_item_expanding.state == (TVIS_SELECTED|TVIS_EXPANDEDONCE), "got state on TVN_ITEMEXPANDING 0x%08x\n", + g_item_expanding.state); + g_get_from_expand = FALSE; + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_ADD, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_kb_seq, "expand node", FALSE); + + /* go to child */ + ret = SendMessageA(hTree, WM_KEYDOWN, VK_RIGHT, 0); + expect(FALSE, ret); + + /* try to expand child that doesn't have children itself */ + flush_sequences(sequences, NUM_MSG_SEQUENCES); + ret = SendMessageA(hTree, WM_KEYDOWN, VK_ADD, 0); + expect(FALSE, ret); + ok_sequence(sequences, PARENT_SEQ_INDEX, parent_expand_empty_kb_seq, "expand node with no children", FALSE); DestroyWindow(hTree); } @@ -1556,6 +1659,55 @@ DestroyWindow(hTree); } +static void test_cchildren(void) +{ + HWND hTree; + INT ret; + TVITEMA item; + + hTree = create_treeview_control(0); + fill_tree(hTree); + + ret = SendMessage(hTree, TVM_DELETEITEM, 0, (LPARAM)hChild); + expect(TRUE, ret); + + /* check cChildren - automatic mode */ + item.hItem = hRoot; + item.mask = TVIF_CHILDREN; + + ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + expect(0, item.cChildren); + + DestroyWindow(hTree); + + /* start over */ + hTree = create_treeview_control(0); + fill_tree(hTree); + + /* turn off automatic mode by setting cChildren explicitly */ + item.hItem = hRoot; + item.mask = TVIF_CHILDREN; + + ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + expect(1, item.cChildren); + + ret = SendMessageA(hTree, TVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + + ret = SendMessage(hTree, TVM_DELETEITEM, 0, (LPARAM)hChild); + expect(TRUE, ret); + + /* check cChildren */ + ret = SendMessageA(hTree, TVM_GETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); +todo_wine + expect(1, item.cChildren); + + DestroyWindow(hTree); +} + struct _ITEM_DATA { HTREEITEM parent; /* for root value of parent field is unidetified */ @@ -1611,9 +1763,9 @@ static void test_TVS_CHECKBOXES(void) { - HIMAGELIST himl; + HIMAGELIST himl, himl2; + HWND hTree, hTree2; TVITEMA item; - HWND hTree; DWORD ret; hTree = create_treeview_control(0); @@ -1643,6 +1795,10 @@ himl = (HIMAGELIST)SendMessageA(hTree, TVM_GETIMAGELIST, TVSIL_STATE, 0); ok(himl != NULL, "got %p\n", himl); + himl2 = (HIMAGELIST)SendMessageA(hTree, TVM_GETIMAGELIST, TVSIL_STATE, 0); + ok(himl2 != NULL, "got %p\n", himl2); + ok(himl2 == himl, "got %p, expected %p\n", himl2, himl); + item.hItem = hRoot; item.mask = TVIF_STATE; item.state = 0; @@ -1659,6 +1815,25 @@ expect(TRUE, ret); ok(item.state == INDEXTOSTATEIMAGEMASK(1), "got 0x%x\n", item.state); + /* create another control and check its checkbox list */ + hTree2 = create_treeview_control(0); + fill_tree(hTree2); + + /* set some index for a child */ + item.hItem = hChild; + item.mask = TVIF_STATE; + item.state = INDEXTOSTATEIMAGEMASK(4); + item.stateMask = TVIS_STATEIMAGEMASK; + ret = SendMessageA(hTree2, TVM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + + /* enabling check boxes set all items to 1 state image index */ + SetWindowLongA(hTree2, GWL_STYLE, GetWindowLongA(hTree, GWL_STYLE) | TVS_CHECKBOXES); + himl2 = (HIMAGELIST)SendMessageA(hTree2, TVM_GETIMAGELIST, TVSIL_STATE, 0); + ok(himl2 != NULL, "got %p\n", himl2); + ok(himl != himl2, "got %p, expected %p\n", himl2, himl); + + DestroyWindow(hTree2); DestroyWindow(hTree); /* the same, but initially created with TVS_CHECKBOXES */ @@ -1726,6 +1901,47 @@ item = (HTREEITEM)SendMessageA(hTree, TVM_GETNEXTITEM, TVGN_PARENT, (LPARAM)hChild); ok(item == hRoot, "got %p, expected %p\n", item, hRoot); + + DestroyWindow(hTree); +} + +static void test_TVM_HITTEST(void) +{ + HWND hTree; + LRESULT ret; + RECT rc; + TVHITTESTINFO ht; + + hTree = create_treeview_control(0); + fill_tree(hTree); + + *(HTREEITEM*)&rc = hRoot; + ret = SendMessage(hTree, TVM_GETITEMRECT, TRUE, (LPARAM)&rc); + expect(TRUE, (BOOL)ret); + + ht.pt.x = rc.left-1; + ht.pt.y = rc.top; + + ret = SendMessage(hTree, TVM_HITTEST, 0, (LPARAM)&ht); + ok((HTREEITEM)ret == hRoot, "got %p, expected %p\n", (HTREEITEM)ret, hRoot); + ok(ht.hItem == hRoot, "got %p, expected %p\n", ht.hItem, hRoot); + ok(ht.flags == TVHT_ONITEMBUTTON, "got %d, expected %d\n", ht.flags, TVHT_ONITEMBUTTON); + + ret = SendMessageA(hTree, TVM_EXPAND, TVE_EXPAND, (LPARAM)hRoot); + expect(TRUE, (BOOL)ret); + + *(HTREEITEM*)&rc = hChild; + ret = SendMessage(hTree, TVM_GETITEMRECT, TRUE, (LPARAM)&rc); + expect(TRUE, (BOOL)ret); + + ht.pt.x = rc.left-1; + ht.pt.y = rc.top; + + ret = SendMessage(hTree, TVM_HITTEST, 0, (LPARAM)&ht); + ok((HTREEITEM)ret == hChild, "got %p, expected %p\n", (HTREEITEM)ret, hChild); + ok(ht.hItem == hChild, "got %p, expected %p\n", ht.hItem, hChild); + /* Wine returns item button here, but this item has no button */ + todo_wine ok(ht.flags == TVHT_ONITEMINDENT, "got %d, expected %d\n", ht.flags, TVHT_ONITEMINDENT); DestroyWindow(hTree); } @@ -1798,9 +2014,11 @@ test_TVS_SINGLEEXPAND(); test_WM_PAINT(); test_delete_items(); + test_cchildren(); test_htreeitem_layout(); test_TVS_CHECKBOXES(); test_TVM_GETNEXTITEM(); + test_TVM_HITTEST(); if (!load_v6_module(&ctx_cookie, &hCtx)) {

1 0

[akhaldi] 57914: [COMCTL32] * Sync with Wine 1.5.19.
by akhaldi＠svn.reactos.org 14 Dec '12

14 Dec '12

Author: akhaldi Date: Fri Dec 14 23:23:44 2012 New Revision: 57914 URL: http://svn.reactos.org/svn/reactos?rev=57914&view=rev Log: [COMCTL32] * Sync with Wine 1.5.19. Modified: trunk/reactos/dll/win32/comctl32/comboex.c trunk/reactos/dll/win32/comctl32/comctl32.h trunk/reactos/dll/win32/comctl32/datetime.c trunk/reactos/dll/win32/comctl32/header.c trunk/reactos/dll/win32/comctl32/idb_hist_large.bmp trunk/reactos/dll/win32/comctl32/idb_hist_small.bmp trunk/reactos/dll/win32/comctl32/idb_std_large.bmp trunk/reactos/dll/win32/comctl32/idb_std_small.bmp trunk/reactos/dll/win32/comctl32/idb_view_large.bmp trunk/reactos/dll/win32/comctl32/idb_view_small.bmp trunk/reactos/dll/win32/comctl32/imagelist.c trunk/reactos/dll/win32/comctl32/listview.c trunk/reactos/dll/win32/comctl32/monthcal.c trunk/reactos/dll/win32/comctl32/pager.c trunk/reactos/dll/win32/comctl32/progress.c trunk/reactos/dll/win32/comctl32/propsheet.c trunk/reactos/dll/win32/comctl32/rebar.c trunk/reactos/dll/win32/comctl32/status.c trunk/reactos/dll/win32/comctl32/string.c trunk/reactos/dll/win32/comctl32/tab.c trunk/reactos/dll/win32/comctl32/toolbar.c trunk/reactos/dll/win32/comctl32/tooltips.c trunk/reactos/dll/win32/comctl32/trackbar.c trunk/reactos/dll/win32/comctl32/treeview.c trunk/reactos/include/psdk/commctrl.h trunk/reactos/media/doc/README.WINE Modified: trunk/reactos/dll/win32/comctl32/comboex.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/comboex… ============================================================================== --- trunk/reactos/dll/win32/comctl32/comboex.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/comboex.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -720,7 +720,7 @@ /* see if we need to change the word break proc on the edit */ if ((infoPtr->dwExtStyle ^ dwTemp) & CBES_EX_PATHWORDBREAKPROC) SetPathWordBreakProc(infoPtr->hwndEdit, - (infoPtr->dwExtStyle & CBES_EX_PATHWORDBREAKPROC) ? TRUE : FALSE); + (infoPtr->dwExtStyle & CBES_EX_PATHWORDBREAKPROC) != 0); /* test if the control's appearance has changed */ mask = CBES_EX_NOEDITIMAGE | CBES_EX_NOEDITIMAGEINDENT; @@ -1381,8 +1381,6 @@ item = infoPtr->edit; if (infoPtr->hwndEdit) { - INT len; - /* free previous text of edit item */ COMBOEX_FreeText(item); item->mask &= ~CBEIF_TEXT; @@ -2076,7 +2074,6 @@ * For EN_CHANGE this issues the same calls and messages * as the native seems to do. */ - WCHAR edit_text[260]; LPCWSTR lastwrk; cmp_func_t cmptext = get_cmp_func(infoPtr); @@ -2311,7 +2308,8 @@ return COMBOEX_WindowPosChanging (infoPtr, (WINDOWPOS *)lParam); case WM_SETFOCUS: - SetFocus(infoPtr->hwndCombo); + if (infoPtr->hwndEdit) SetFocus( infoPtr->hwndEdit ); + else SetFocus( infoPtr->hwndCombo ); return 0; case WM_SYSCOLORCHANGE: Modified: trunk/reactos/dll/win32/comctl32/comctl32.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/comctl3… ============================================================================== --- trunk/reactos/dll/win32/comctl32/comctl32.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/comctl32.h [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -231,6 +231,7 @@ int MONTHCAL_MonthLength(int month, int year) DECLSPEC_HIDDEN; int MONTHCAL_CalculateDayOfWeek(SYSTEMTIME *date, BOOL inplace) DECLSPEC_HIDDEN; +LONG MONTHCAL_CompareSystemTime(const SYSTEMTIME *first, const SYSTEMTIME *second) DECLSPEC_HIDDEN; extern void THEMING_Initialize(void) DECLSPEC_HIDDEN; extern void THEMING_Uninitialize(void) DECLSPEC_HIDDEN; Modified: trunk/reactos/dll/win32/comctl32/datetime.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/datetim… ============================================================================== --- trunk/reactos/dll/win32/comctl32/datetime.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/datetime.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -4,6 +4,7 @@ * Copyright 1998, 1999 Eric Kohl * Copyright 1999, 2000 Alex Priem <alexp(a)sci.kun.nl> * Copyright 2000 Chris Morgan <cmorgan(a)wpi.edu> + * Copyright 2012 Owen Rudge for CodeWeavers * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -138,6 +139,9 @@ static const WCHAR allowedformatchars[] = {'d', 'h', 'H', 'm', 'M', 's', 't', 'y', 'X', 0}; static const int maxrepetition [] = {4,2,2,2,4,2,2,4,-1}; +/* valid date limits */ +static const SYSTEMTIME max_allowed_date = { /* wYear */ 9999, /* wMonth */ 12, /* wDayOfWeek */ 0, /* wDay */ 31 }; +static const SYSTEMTIME min_allowed_date = { /* wYear */ 1752, /* wMonth */ 9, /* wDayOfWeek */ 0, /* wDay */ 14 }; static DWORD DATETIME_GetSystemTime (const DATETIME_INFO *infoPtr, SYSTEMTIME *systime) @@ -153,6 +157,43 @@ return GDT_VALID; } +/* Checks value is within configured date range + * + * PARAMETERS + * + * [I] infoPtr : valid pointer to control data + * [I] date : pointer to valid date data to check + * + * RETURN VALUE + * + * TRUE - date within configured range + * FALSE - date is outside configured range + */ +static BOOL DATETIME_IsDateInValidRange(const DATETIME_INFO *infoPtr, const SYSTEMTIME *date) +{ + SYSTEMTIME range[2]; + DWORD limits; + + if ((MONTHCAL_CompareSystemTime(date, &max_allowed_date) == 1) || + (MONTHCAL_CompareSystemTime(date, &min_allowed_date) == -1)) + return FALSE; + + limits = SendMessageW (infoPtr->hMonthCal, MCM_GETRANGE, 0, (LPARAM)range); + + if (limits & GDTR_MAX) + { + if (MONTHCAL_CompareSystemTime(date, &range[1]) == 1) + return FALSE; + } + + if (limits & GDTR_MIN) + { + if (MONTHCAL_CompareSystemTime(date, &range[0]) == -1) + return FALSE; + } + + return TRUE; +} static BOOL DATETIME_SetSystemTime (DATETIME_INFO *infoPtr, DWORD flag, const SYSTEMTIME *systime) @@ -164,7 +205,7 @@ systime->wHour, systime->wMinute, systime->wSecond); if (flag == GDT_VALID) { - if (systime->wYear < 1601 || systime->wYear > 30827 || + if (systime->wYear == 0 || systime->wMonth < 1 || systime->wMonth > 12 || systime->wDay < 1 || systime->wDay > MONTHCAL_MonthLength(systime->wMonth, systime->wYear) || @@ -174,6 +215,10 @@ systime->wMilliseconds > 999 ) return FALSE; + + /* Windows returns true if the date is valid but outside the limits set */ + if (DATETIME_IsDateInValidRange(infoPtr, systime) == FALSE) + return TRUE; infoPtr->dateValid = TRUE; infoPtr->date = *systime; @@ -462,6 +507,9 @@ DATETIME_IncreaseField (DATETIME_INFO *infoPtr, int number, int delta) { SYSTEMTIME *date = &infoPtr->date; + SYSTEMTIME range[2]; + DWORD limits; + BOOL min; TRACE ("%d\n", number); if ((number > infoPtr->nrFields) || (number < 0)) return; @@ -472,7 +520,13 @@ case ONEDIGITYEAR: case TWODIGITYEAR: case FULLYEAR: - date->wYear = wrap(date->wYear, delta, 1752, 9999); + if (delta == INT_MIN) + date->wYear = 1752; + else if (delta == INT_MAX) + date->wYear = 9999; + else + date->wYear = max(min(date->wYear + delta, 9999), 1752); + if (date->wDay > MONTHCAL_MonthLength(date->wMonth, date->wYear)) /* This can happen when moving away from a leap year. */ date->wDay = MONTHCAL_MonthLength(date->wMonth, date->wYear); @@ -529,8 +583,28 @@ date->wMinute = 0; date->wHour = 0; } -} - + + /* Ensure time is within bounds */ + limits = SendMessageW (infoPtr->hMonthCal, MCM_GETRANGE, 0, (LPARAM)range); + min = delta < 0; + + if (limits & (min ? GDTR_MIN : GDTR_MAX)) + { + int i = (min ? 0 : 1); + + if (MONTHCAL_CompareSystemTime(date, &range[i]) == (min ? -1 : 1)) + { + date->wYear = range[i].wYear; + date->wMonth = range[i].wMonth; + date->wDayOfWeek = range[i].wDayOfWeek; + date->wDay = range[i].wDay; + date->wHour = range[i].wHour; + date->wMinute = range[i].wMinute; + date->wSecond = range[i].wSecond; + date->wMilliseconds = range[i].wMilliseconds; + } + } +} static void DATETIME_ReturnFieldWidth (const DATETIME_INFO *infoPtr, HDC hdc, int count, SHORT *width) @@ -774,25 +848,43 @@ int fieldNum = infoPtr->select & DTHT_DATEFIELD; int i, val=0, clamp_day=0; SYSTEMTIME date = infoPtr->date; + int oldyear; if (infoPtr->select == -1 || infoPtr->nCharsEntered == 0) return; - for (i=0; i<infoPtr->nCharsEntered; i++) - val = val * 10 + infoPtr->charsEntered[i] - '0'; + if ((infoPtr->fieldspec[fieldNum] == ONELETTERAMPM) || + (infoPtr->fieldspec[fieldNum] == TWOLETTERAMPM)) + val = infoPtr->charsEntered[0]; + else { + for (i=0; i<infoPtr->nCharsEntered; i++) + val = val * 10 + infoPtr->charsEntered[i] - '0'; + } infoPtr->nCharsEntered = 0; switch (infoPtr->fieldspec[fieldNum]) { case ONEDIGITYEAR: case TWODIGITYEAR: + oldyear = date.wYear; date.wYear = date.wYear - (date.wYear%100) + val; - clamp_day = 1; + + if (DATETIME_IsDateInValidRange(infoPtr, &date)) + clamp_day = 1; + else + date.wYear = oldyear; + break; case INVALIDFULLYEAR: case FULLYEAR: + oldyear = date.wYear; date.wYear = val; - clamp_day = 1; + + if (DATETIME_IsDateInValidRange(infoPtr, &date)) + clamp_day = 1; + else + date.wYear = oldyear; + break; case ONEDIGITMONTH: case TWODIGITMONTH: @@ -805,9 +897,20 @@ break; case ONEDIGIT12HOUR: case TWODIGIT12HOUR: + if (val >= 24) + val -= 20; + + if (val >= 13) + date.wHour = val; + else if (val != 0) { + if (date.wHour >= 12) /* preserve current AM/PM state */ + date.wHour = (val == 12 ? 12 : val + 12); + else + date.wHour = (val == 12 ? 0 : val); + } + break; case ONEDIGIT24HOUR: case TWODIGIT24HOUR: - /* FIXME: Preserve AM/PM for 12HOUR? */ date.wHour = val; break; case ONEDIGITMINUTE: @@ -817,6 +920,16 @@ case ONEDIGITSECOND: case TWODIGITSECOND: date.wSecond = val; + break; + case ONELETTERAMPM: + case TWOLETTERAMPM: + if (val == 'a' || val == 'A') { + if (date.wHour >= 12) + date.wHour -= 12; + } else if (val == 'p' || val == 'P') { + if (date.wHour < 12) + date.wHour += 12; + } break; } @@ -1111,24 +1224,40 @@ static LRESULT DATETIME_Char (DATETIME_INFO *infoPtr, WPARAM vkCode) { - int fieldNum = infoPtr->select & DTHT_DATEFIELD; - - if (vkCode >= '0' && vkCode <= '9') { + int fieldNum, fieldSpec; + + fieldNum = infoPtr->select & DTHT_DATEFIELD; + fieldSpec = infoPtr->fieldspec[fieldNum]; + + if (fieldSpec == ONELETTERAMPM || fieldSpec == TWOLETTERAMPM) { + infoPtr->charsEntered[0] = vkCode; + infoPtr->nCharsEntered = 1; + + DATETIME_ApplySelectedField(infoPtr); + } else if (vkCode >= '0' && vkCode <= '9') { int maxChars; - int fieldSpec; infoPtr->charsEntered[infoPtr->nCharsEntered++] = vkCode; - - fieldSpec = infoPtr->fieldspec[fieldNum]; if (fieldSpec == INVALIDFULLYEAR || fieldSpec == FULLYEAR) maxChars = 4; else maxChars = 2; + if ((fieldSpec == ONEDIGIT12HOUR || + fieldSpec == TWODIGIT12HOUR || + fieldSpec == ONEDIGIT24HOUR || + fieldSpec == TWODIGIT24HOUR) && + (infoPtr->nCharsEntered == 1)) + { + if (vkCode >= '3') + maxChars = 1; + } + if (maxChars == infoPtr->nCharsEntered) DATETIME_ApplySelectedField(infoPtr); } + return 0; } @@ -1316,7 +1445,7 @@ infoPtr->hwndCheckbut = CreateWindowExW (0, WC_BUTTONW, 0, WS_CHILD | WS_VISIBLE | BS_AUTOCHECKBOX, 2, 2, 13, 13, infoPtr->hwndSelf, 0, (HINSTANCE)GetWindowLongPtrW (infoPtr->hwndSelf, GWLP_HINSTANCE), 0); - SendMessageW (infoPtr->hwndCheckbut, BM_SETCHECK, 1, 0); + SendMessageW (infoPtr->hwndCheckbut, BM_SETCHECK, infoPtr->dateValid ? 1 : 0, 0); } if ( (lpss->styleOld & DTS_SHOWNONE) && !(lpss->styleNew & DTS_SHOWNONE) ) { DestroyWindow(infoPtr->hwndCheckbut); Modified: trunk/reactos/dll/win32/comctl32/header.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/header.… ============================================================================== --- trunk/reactos/dll/win32/comctl32/header.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/header.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -727,17 +727,9 @@ HEADER_DrawTrackLine (const HEADER_INFO *infoPtr, HDC hdc, INT x) { RECT rect; - HPEN hOldPen; - INT oldRop; GetClientRect (infoPtr->hwndSelf, &rect); - - hOldPen = SelectObject (hdc, GetStockObject (BLACK_PEN)); - oldRop = SetROP2 (hdc, R2_XORPEN); - MoveToEx (hdc, x, rect.top, NULL); - LineTo (hdc, x, rect.bottom); - SetROP2 (hdc, oldRop); - SelectObject (hdc, hOldPen); + PatBlt( hdc, x, rect.top, 1, rect.bottom - rect.top, DSTINVERT ); } /*** @@ -1240,6 +1232,14 @@ if ((UINT)size != infoPtr->uNumItem) return FALSE; + + if (TRACE_ON(header)) + { + TRACE("count=%d, order array={", size); + for (i = 0; i < size; i++) + TRACE("%d%c", order[i], i != size-1 ? ',' : '}'); + TRACE("\n"); + } for (i=0; i<size; i++) { Modified: trunk/reactos/dll/win32/comctl32/idb_hist_large.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_his… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/idb_hist_small.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_his… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/idb_std_large.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_std… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/idb_std_small.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_std… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/idb_view_large.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_vie… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/idb_view_small.bmp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/idb_vie… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/dll/win32/comctl32/imagelist.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/imageli… ============================================================================== --- trunk/reactos/dll/win32/comctl32/imagelist.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/imagelist.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -1186,7 +1186,7 @@ /************************************************************************* * ImageList_DrawEx [COMCTL32.@] * - * Draws an image and allows to use extended drawing features. + * Draws an image and allows using extended drawing features. * * PARAMS * himl [I] handle to image list @@ -3366,7 +3366,7 @@ return E_FAIL; /* TODO: Add test for IID_ImageList2 too */ - if (FAILED(IImageList_QueryInterface(punkSrc, &IID_IImageList, + if (FAILED(IUnknown_QueryInterface(punkSrc, &IID_IImageList, (void **) &src))) return E_FAIL; @@ -3390,7 +3390,7 @@ TRACE("(%p)->(%d %p %d %d %d %s %p)\n", iface, i1, punk2, i2, dx, dy, debugstr_guid(riid), ppv); /* TODO: Add test for IID_ImageList2 too */ - if (FAILED(IImageList_QueryInterface(punk2, &IID_IImageList, + if (FAILED(IUnknown_QueryInterface(punk2, &IID_IImageList, (void **) &iml2))) return E_FAIL; @@ -3523,7 +3523,7 @@ return E_FAIL; /* TODO: Add test for IID_ImageList2 too */ - if (FAILED(IImageList_QueryInterface(punk, &IID_IImageList, + if (FAILED(IUnknown_QueryInterface(punk, &IID_IImageList, (void **) &iml2))) return E_FAIL; Modified: trunk/reactos/dll/win32/comctl32/listview.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/listvie… ============================================================================== --- trunk/reactos/dll/win32/comctl32/listview.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/listview.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -6,7 +6,7 @@ * Copyright 2000 Jason Mawdsley * Copyright 2001 CodeWeavers Inc. * Copyright 2002 Dimitrie O. Paun - * Copyright 2009-2011 Nikolay Sivov + * Copyright 2009-2012 Nikolay Sivov * Copyright 2009 Owen Rudge for CodeWeavers * * This library is free software; you can redistribute it and/or @@ -560,11 +560,8 @@ static inline int lstrncmpiW(LPCWSTR s1, LPCWSTR s2, int n) { - int res; - n = min(min(n, lstrlenW(s1)), lstrlenW(s2)); - res = CompareStringW(LOCALE_USER_DEFAULT, NORM_IGNORECASE, s1, n, s2, n); - return res ? res - sizeof(WCHAR) : res; + return CompareStringW(LOCALE_USER_DEFAULT, NORM_IGNORECASE, s1, n, s2, n) - CSTR_EQUAL; } /******** Debugging functions *****************************************/ @@ -3901,7 +3898,7 @@ } iterator_destroy(&new_elems); - LISTVIEW_InvalidateRect(infoPtr, &rect); + LISTVIEW_InvalidateRect(infoPtr, &infoPtr->marqueeDrawRect); } /*** @@ -4288,6 +4285,10 @@ { if (lpLVItem->state & LVIS_FOCUSED) { + /* update selection mark */ + if (infoPtr->nFocusedItem == -1 && infoPtr->nSelectionMark == -1) + infoPtr->nSelectionMark = lpLVItem->iItem; + if (infoPtr->nFocusedItem != -1) { /* remove current focus */ @@ -4957,10 +4958,7 @@ itemheight = LISTVIEW_CalculateItemHeight(infoPtr); rcItem.left = infoPtr->rcList.left; rcItem.right = infoPtr->rcList.right; - rcItem.bottom = rcItem.top = Origin.y - 1; - MoveToEx(hdc, rcItem.left, rcItem.top, NULL); - LineTo(hdc, rcItem.right, rcItem.top); - for(y=itemheight-1+Origin.y; y<=infoPtr->rcList.bottom; y+=itemheight) + for(y = Origin.y > 1 ? Origin.y - 1 : itemheight - 1 + Origin.y % itemheight; y<=infoPtr->rcList.bottom; y+=itemheight) { rcItem.bottom = rcItem.top = y; TRACE("horz rcItem=%s\n", wine_dbgstr_rect(&rcItem)); @@ -8584,10 +8582,12 @@ if (cy == -1) cy = GetSystemMetrics(SM_CYICONSPACING); /* if 0 then compute width - * FIXME: Should scan each item and determine max width of - * icon or label, then make that the width */ - if (cx == 0) - cx = infoPtr->iconSpacing.cx; + * FIXME: computed cx and cy is not matching native behaviour */ + if (cx == 0) { + cx = GetSystemMetrics(SM_CXICONSPACING); + if (infoPtr->iconSize.cx + ICON_LR_PADDING > cx) + cx = infoPtr->iconSize.cx + ICON_LR_PADDING; + } /* if 0 then compute height */ if (cy == 0) @@ -8829,26 +8829,31 @@ * PARAMETER(S): * [I] infoPtr : valid pointer to the listview structure * [I] nItem : item index - * [I] lpLVItem : item or subitem info + * [I] item : item or subitem info * * RETURN: * SUCCESS : TRUE * FAILURE : FALSE */ -static BOOL LISTVIEW_SetItemState(LISTVIEW_INFO *infoPtr, INT nItem, const LVITEMW *lpLVItem) -{ - BOOL bResult = TRUE; +static BOOL LISTVIEW_SetItemState(LISTVIEW_INFO *infoPtr, INT nItem, const LVITEMW *item) +{ + BOOL ret = TRUE; LVITEMW lvItem; + + if (!item) return FALSE; lvItem.iItem = nItem; lvItem.iSubItem = 0; lvItem.mask = LVIF_STATE; - lvItem.state = lpLVItem->state; - lvItem.stateMask = lpLVItem->stateMask; - TRACE("lvItem=%s\n", debuglvitem_t(&lvItem, TRUE)); + lvItem.state = item->state; + lvItem.stateMask = item->stateMask; + TRACE("item=%s\n", debuglvitem_t(&lvItem, TRUE)); if (nItem == -1) { + UINT oldstate = 0; + BOOL notify; + /* select all isn't allowed in LVS_SINGLESEL */ if ((lvItem.state & lvItem.stateMask & LVIS_SELECTED) && (infoPtr->dwStyle & LVS_SINGLESEL)) return FALSE; @@ -8856,14 +8861,40 @@ /* focus all isn't allowed */ if (lvItem.state & lvItem.stateMask & LVIS_FOCUSED) return FALSE; + notify = infoPtr->bDoChangeNotify; + if (infoPtr->dwStyle & LVS_OWNERDATA) + { + infoPtr->bDoChangeNotify = FALSE; + if (!(lvItem.state & LVIS_SELECTED) && LISTVIEW_GetSelectedCount(infoPtr)) + oldstate |= LVIS_SELECTED; + if (infoPtr->nFocusedItem != -1) oldstate |= LVIS_FOCUSED; + } + /* apply to all items */ for (lvItem.iItem = 0; lvItem.iItem < infoPtr->nItemCount; lvItem.iItem++) - if (!LISTVIEW_SetItemT(infoPtr, &lvItem, TRUE)) bResult = FALSE; + if (!LISTVIEW_SetItemT(infoPtr, &lvItem, TRUE)) ret = FALSE; + + if (infoPtr->dwStyle & LVS_OWNERDATA) + { + NMLISTVIEW nmlv; + + infoPtr->bDoChangeNotify = notify; + + nmlv.iItem = -1; + nmlv.iSubItem = 0; + nmlv.uNewState = lvItem.state & lvItem.stateMask; + nmlv.uOldState = oldstate & lvItem.stateMask; + nmlv.uChanged = LVIF_STATE; + nmlv.ptAction.x = nmlv.ptAction.y = 0; + nmlv.lParam = 0; + + notify_listview(infoPtr, LVN_ITEMCHANGED, &nmlv); + } } else - bResult = LISTVIEW_SetItemT(infoPtr, &lvItem, TRUE); - - return bResult; + ret = LISTVIEW_SetItemT(infoPtr, &lvItem, TRUE); + + return ret; } /*** @@ -8885,6 +8916,7 @@ LVITEMW lvItem; if (!lpLVItem || nItem < 0 || nItem >= infoPtr->nItemCount) return FALSE; + if (infoPtr->dwStyle & LVS_OWNERDATA) return FALSE; lvItem.iItem = nItem; lvItem.iSubItem = lpLVItem->iSubItem; @@ -9252,21 +9284,15 @@ */ static BOOL LISTVIEW_DrawTrackLine(const LISTVIEW_INFO *infoPtr) { - HPEN hOldPen; HDC hdc; - INT oldROP; if (infoPtr->xTrackLine == -1) return FALSE; if (!(hdc = GetDC(infoPtr->hwndSelf))) return FALSE; - hOldPen = SelectObject(hdc, GetStockObject(BLACK_PEN)); - oldROP = SetROP2(hdc, R2_XORPEN); - MoveToEx(hdc, infoPtr->xTrackLine, infoPtr->rcList.top, NULL); - LineTo(hdc, infoPtr->xTrackLine, infoPtr->rcList.bottom); - SetROP2(hdc, oldROP); - SelectObject(hdc, hOldPen); + PatBlt( hdc, infoPtr->xTrackLine, infoPtr->rcList.top, + 1, infoPtr->rcList.bottom - infoPtr->rcList.top, DSTINVERT ); ReleaseDC(infoPtr->hwndSelf, hdc); return TRUE; } @@ -9418,6 +9444,7 @@ /* init item size to avoid division by 0 */ LISTVIEW_UpdateItemSize (infoPtr); + LISTVIEW_UpdateSize (infoPtr); if (infoPtr->uView == LV_VIEW_DETAILS) { @@ -10873,7 +10900,7 @@ * The "2" is there to mimic the native control. I think it may be * related to either padding or edges. (GLA 7/2002) */ - if (!(infoPtr->dwStyle & WS_HSCROLL)) + if (!(GetWindowLongW(infoPtr->hwndSelf, GWL_STYLE) & WS_HSCROLL)) infoPtr->rcList.bottom -= GetSystemMetrics(SM_CYHSCROLL); infoPtr->rcList.bottom = max (infoPtr->rcList.bottom - 2, 0); } @@ -11431,7 +11458,6 @@ return LISTVIEW_SetItemPosition(infoPtr, (INT)wParam, (POINT*)lParam); case LVM_SETITEMSTATE: - if (lParam == 0) return FALSE; return LISTVIEW_SetItemState(infoPtr, (INT)wParam, (LPLVITEMW)lParam); case LVM_SETITEMTEXTA: @@ -11580,9 +11606,6 @@ case WM_SHOWWINDOW: return LISTVIEW_ShowWindow(infoPtr, wParam, lParam); - case WM_SIZE: - return LISTVIEW_Size(infoPtr, (short)LOWORD(lParam), (short)HIWORD(lParam)); - case WM_STYLECHANGED: return LISTVIEW_StyleChanged(infoPtr, wParam, (LPSTYLESTRUCT)lParam); @@ -11614,16 +11637,14 @@ case WM_WINDOWPOSCHANGED: if (!(((WINDOWPOS *)lParam)->flags & SWP_NOSIZE)) { - SetWindowPos(infoPtr->hwndSelf, 0, 0, 0, 0, 0, SWP_FRAMECHANGED | SWP_NOACTIVATE | - SWP_NOZORDER | SWP_NOMOVE | SWP_NOSIZE); - - if ((infoPtr->dwStyle & LVS_OWNERDRAWFIXED) && (infoPtr->uView == LV_VIEW_DETAILS)) - { - if (notify_measureitem(infoPtr)) LISTVIEW_InvalidateList(infoPtr); - } - - LISTVIEW_UpdateSize(infoPtr); - LISTVIEW_UpdateScroll(infoPtr); + SetWindowPos(infoPtr->hwndSelf, 0, 0, 0, 0, 0, SWP_FRAMECHANGED | SWP_NOACTIVATE | + SWP_NOZORDER | SWP_NOMOVE | SWP_NOSIZE); + + if ((infoPtr->dwStyle & LVS_OWNERDRAWFIXED) && (infoPtr->uView == LV_VIEW_DETAILS)) + { + if (notify_measureitem(infoPtr)) LISTVIEW_InvalidateList(infoPtr); + } + LISTVIEW_Size(infoPtr, ((WINDOWPOS *)lParam)->cx, ((WINDOWPOS *)lParam)->cy); } return DefWindowProcW(hwnd, uMsg, wParam, lParam); Modified: trunk/reactos/dll/win32/comctl32/monthcal.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/monthca… ============================================================================== --- trunk/reactos/dll/win32/comctl32/monthcal.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/monthcal.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -185,7 +185,14 @@ nmsc.nmhdr.idFrom = GetWindowLongPtrW(infoPtr->hwndSelf, GWLP_ID); nmsc.nmhdr.code = MCN_SELCHANGE; nmsc.stSelStart = infoPtr->minSel; - nmsc.stSelEnd = infoPtr->maxSel; + nmsc.stSelStart.wDayOfWeek = 0; + if(infoPtr->dwStyle & MCS_MULTISELECT){ + nmsc.stSelEnd = infoPtr->maxSel; + nmsc.stSelEnd.wDayOfWeek = 0; + } + else + nmsc.stSelEnd = st_null; + SendMessageW(infoPtr->hwndNotify, WM_NOTIFY, nmsc.nmhdr.idFrom, (LPARAM)&nmsc); } @@ -198,7 +205,13 @@ nmsc.nmhdr.idFrom = GetWindowLongPtrW(infoPtr->hwndSelf, GWLP_ID); nmsc.nmhdr.code = MCN_SELECT; nmsc.stSelStart = infoPtr->minSel; - nmsc.stSelEnd = infoPtr->maxSel; + nmsc.stSelStart.wDayOfWeek = 0; + if(infoPtr->dwStyle & MCS_MULTISELECT){ + nmsc.stSelEnd = infoPtr->maxSel; + nmsc.stSelEnd.wDayOfWeek = 0; + } + else + nmsc.stSelEnd = st_null; SendMessageW(infoPtr->hwndNotify, WM_NOTIFY, nmsc.nmhdr.idFrom, (LPARAM)&nmsc); } @@ -296,7 +309,7 @@ * * Note that no date validation performed, already validated values expected. */ -static LONG MONTHCAL_CompareSystemTime(const SYSTEMTIME *first, const SYSTEMTIME *second) +LONG MONTHCAL_CompareSystemTime(const SYSTEMTIME *first, const SYSTEMTIME *second) { FILETIME ft_first, ft_second; @@ -353,14 +366,17 @@ else if(MONTHCAL_CompareSystemTime(date, &min_allowed_date) == -1) { fix_st = &min_allowed_date; } - else if(infoPtr->rangeValid & GDTR_MAX) { - if((MONTHCAL_CompareSystemTime(date, &infoPtr->maxDate) == 1)) { - fix_st = &infoPtr->maxDate; + else { + if(infoPtr->rangeValid & GDTR_MAX) { + if((MONTHCAL_CompareSystemTime(date, &infoPtr->maxDate) == 1)) { + fix_st = &infoPtr->maxDate; + } } - } - else if(infoPtr->rangeValid & GDTR_MIN) { - if((MONTHCAL_CompareSystemTime(date, &infoPtr->minDate) == -1)) { - fix_st = &infoPtr->minDate; + + if(infoPtr->rangeValid & GDTR_MIN) { + if((MONTHCAL_CompareSystemTime(date, &infoPtr->minDate) == -1)) { + fix_st = &infoPtr->minDate; + } } } @@ -369,7 +385,7 @@ date->wMonth = fix_st->wMonth; } - return fix_st ? FALSE : TRUE; + return !fix_st; } /* Checks passed range width with configured maximum selection count @@ -1273,7 +1289,7 @@ infoPtr->pens[PenText] = CreatePen(PS_SOLID, 1, infoPtr->colors[index]); } - InvalidateRect(infoPtr->hwndSelf, NULL, index == MCSC_BACKGROUND ? TRUE : FALSE); + InvalidateRect(infoPtr->hwndSelf, NULL, index == MCSC_BACKGROUND); return prev; } Modified: trunk/reactos/dll/win32/comctl32/pager.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/pager.c… ============================================================================== --- trunk/reactos/dll/win32/comctl32/pager.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/pager.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -84,9 +84,6 @@ INT direction; /* direction of the scroll, (e.g. PGF_SCROLLUP) */ } PAGER_INFO; -#define MIN_ARROW_WIDTH 8 -#define MIN_ARROW_HEIGHT 5 - #define TIMERID1 1 #define TIMERID2 2 #define INITIAL_DELAY 500 @@ -116,115 +113,13 @@ } } -/* the horizontal arrows are: - * - * 01234 01234 - * 1 * * - * 2 ** ** - * 3*** *** - * 4*** *** - * 5 ** ** - * 6 * * - * 7 - * - */ static void -PAGER_DrawHorzArrow (HDC hdc, RECT r, INT colorRef, BOOL left) -{ - INT x, y, w, h; - HPEN hPen, hOldPen; - - w = r.right - r.left + 1; - h = r.bottom - r.top + 1; - if ((h < MIN_ARROW_WIDTH) || (w < MIN_ARROW_HEIGHT)) - return; /* refuse to draw partial arrow */ - - if (!(hPen = CreatePen( PS_SOLID, 1, GetSysColor( colorRef )))) return; - hOldPen = SelectObject ( hdc, hPen ); - if (left) - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 3; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x--, y+5); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x--, y+3); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x, y+1); - } - else - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x++, y+5); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x++, y+3); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x, y+1); - } - - SelectObject( hdc, hOldPen ); - DeleteObject( hPen ); -} - -/* the vertical arrows are: - * - * 01234567 01234567 - * 1****** ** - * 2 **** **** - * 3 ** ****** - * 4 - * - */ -static void -PAGER_DrawVertArrow (HDC hdc, RECT r, INT colorRef, BOOL up) -{ - INT x, y, w, h; - HPEN hPen, hOldPen; - - w = r.right - r.left + 1; - h = r.bottom - r.top + 1; - if ((h < MIN_ARROW_WIDTH) || (w < MIN_ARROW_HEIGHT)) - return; /* refuse to draw partial arrow */ - - if (!(hPen = CreatePen( PS_SOLID, 1, GetSysColor( colorRef )))) return; - hOldPen = SelectObject ( hdc, hPen ); - if (up) - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 3; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+5, y--); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+3, y--); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+1, y); - } - else - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+5, y++); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+3, y++); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+1, y); - } - - SelectObject( hdc, hOldPen ); - DeleteObject( hPen ); -} - -static void -PAGER_DrawButton(HDC hdc, COLORREF clrBk, RECT arrowRect, +PAGER_DrawButton(HDC hdc, COLORREF clrBk, RECT rc, BOOL horz, BOOL topLeft, INT btnState) { - HBRUSH hBrush, hOldBrush; - RECT rc = arrowRect; - - TRACE("arrowRect = %s, btnState = %d\n", wine_dbgstr_rect(&arrowRect), btnState); + UINT flags; + + TRACE("rc = %s, btnState = %d\n", wine_dbgstr_rect(&rc), btnState); if (btnState == PGF_INVISIBLE) return; @@ -232,54 +127,26 @@ if ((rc.right - rc.left <= 0) || (rc.bottom - rc.top <= 0)) return; - hBrush = CreateSolidBrush(clrBk); - hOldBrush = SelectObject(hdc, hBrush); - - FillRect(hdc, &rc, hBrush); - - if (btnState == PGF_HOT) - { - DrawEdge( hdc, &rc, BDR_RAISEDINNER, BF_RECT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_NORMAL) - { - DrawEdge (hdc, &rc, BDR_OUTER, BF_FLAT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_DEPRESSED) - { - DrawEdge( hdc, &rc, BDR_SUNKENOUTER, BF_RECT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_GRAYED) - { - DrawEdge (hdc, &rc, BDR_OUTER, BF_FLAT); - if (horz) - { - PAGER_DrawHorzArrow(hdc, rc, COLOR_3DHIGHLIGHT, topLeft); - rc.left++, rc.top++; rc.right++, rc.bottom++; - PAGER_DrawHorzArrow(hdc, rc, COLOR_3DSHADOW, topLeft); - } - else - { - PAGER_DrawVertArrow(hdc, rc, COLOR_3DHIGHLIGHT, topLeft); - rc.left++, rc.top++; rc.right++, rc.bottom++; - PAGER_DrawVertArrow(hdc, rc, COLOR_3DSHADOW, topLeft); - } - } - - SelectObject( hdc, hOldBrush ); - DeleteObject(hBrush); + if (horz) + flags = topLeft ? DFCS_SCROLLLEFT : DFCS_SCROLLRIGHT; + else + flags = topLeft ? DFCS_SCROLLUP : DFCS_SCROLLDOWN; + + switch (btnState) + { + case PGF_HOT: + break; + case PGF_NORMAL: + flags |= DFCS_FLAT; + break; + case PGF_DEPRESSED: + flags |= DFCS_PUSHED; + break; + case PGF_GRAYED: + flags |= DFCS_INACTIVE | DFCS_FLAT; + break; + } + DrawFrameControl( hdc, &rc, DFC_SCROLL, flags ); } /* << PAGER_GetDropTarget >> */ @@ -338,22 +205,24 @@ } static void -PAGER_CalcSize (const PAGER_INFO *infoPtr, INT* size, BOOL getWidth) +PAGER_CalcSize( PAGER_INFO *infoPtr ) { NMPGCALCSIZE nmpgcs; ZeroMemory (&nmpgcs, sizeof (NMPGCALCSIZE)); nmpgcs.hdr.hwndFrom = infoPtr->hwndSelf; nmpgcs.hdr.idFrom = GetWindowLongPtrW (infoPtr->hwndSelf, GWLP_ID); nmpgcs.hdr.code = PGN_CALCSIZE; - nmpgcs.dwFlag = getWidth ? PGF_CALCWIDTH : PGF_CALCHEIGHT; - nmpgcs.iWidth = getWidth ? *size : 0; - nmpgcs.iHeight = getWidth ? 0 : *size; + nmpgcs.dwFlag = (infoPtr->dwStyle & PGS_HORZ) ? PGF_CALCWIDTH : PGF_CALCHEIGHT; + nmpgcs.iWidth = infoPtr->nWidth; + nmpgcs.iHeight = infoPtr->nHeight; SendMessageW (infoPtr->hwndNotify, WM_NOTIFY, nmpgcs.hdr.idFrom, (LPARAM)&nmpgcs); - *size = getWidth ? nmpgcs.iWidth : nmpgcs.iHeight; - - TRACE("[%p] PGN_CALCSIZE returns %s=%d\n", infoPtr->hwndSelf, - getWidth ? "width" : "height", *size); + if (infoPtr->dwStyle & PGS_HORZ) + infoPtr->nWidth = nmpgcs.iWidth; + else + infoPtr->nHeight = nmpgcs.iHeight; + + TRACE("[%p] PGN_CALCSIZE returns %dx%d\n", infoPtr->hwndSelf, nmpgcs.iWidth, nmpgcs.iHeight ); } static void @@ -414,16 +283,15 @@ RECT wndRect; GetWindowRect(infoPtr->hwndSelf, &wndRect); + PAGER_CalcSize(infoPtr); if (infoPtr->dwStyle & PGS_HORZ) { wndSize = wndRect.right - wndRect.left; - PAGER_CalcSize(infoPtr, &infoPtr->nWidth, TRUE); childSize = infoPtr->nWidth; } else { wndSize = wndRect.bottom - wndRect.top; - PAGER_CalcSize(infoPtr, &infoPtr->nHeight, FALSE); childSize = infoPtr->nHeight; } @@ -447,9 +315,10 @@ RECT rcTopLeft, rcBottomRight; /* get button rects */ - PAGER_GetButtonRects(infoPtr, &rcTopLeft, &rcBottomRight, FALSE); + PAGER_GetButtonRects(infoPtr, &rcTopLeft, &rcBottomRight, TRUE); GetCursorPos(&pt); + ScreenToClient( infoPtr->hwndSelf, &pt ); /* update states based on scroll position */ if (infoPtr->nPos > 0) @@ -457,7 +326,7 @@ if (infoPtr->TLbtnState == PGF_INVISIBLE || infoPtr->TLbtnState == PGF_GRAYED) infoPtr->TLbtnState = PGF_NORMAL; } - else if (PtInRect(&rcTopLeft, pt)) + else if (!hideGrayBtns && PtInRect(&rcTopLeft, pt)) infoPtr->TLbtnState = PGF_GRAYED; else infoPtr->TLbtnState = PGF_INVISIBLE; @@ -472,7 +341,7 @@ if (infoPtr->BRbtnState == PGF_INVISIBLE || infoPtr->BRbtnState == PGF_GRAYED) infoPtr->BRbtnState = PGF_NORMAL; } - else if (PtInRect(&rcBottomRight, pt)) + else if (!hideGrayBtns && PtInRect(&rcBottomRight, pt)) infoPtr->BRbtnState = PGF_GRAYED; else infoPtr->BRbtnState = PGF_INVISIBLE; @@ -547,68 +416,6 @@ return DefWindowProcW (infoPtr->hwndSelf, WM_WINDOWPOSCHANGING, 0, (LPARAM)winpos); } -static INT -PAGER_SetFixedWidth(PAGER_INFO* infoPtr) -{ - /* Must set the non-scrollable dimension to be less than the full height/width - * so that NCCalcSize is called. The Microsoft docs mention 3/4 factor for button - * size, and experimentation shows that the effect is almost right. */ - - RECT wndRect; - INT delta, h; - GetWindowRect(infoPtr->hwndSelf, &wndRect); - - /* see what the app says for btn width */ - PAGER_CalcSize(infoPtr, &infoPtr->nWidth, TRUE); - - if (infoPtr->dwStyle & CCS_NORESIZE) - { - delta = wndRect.right - wndRect.left - infoPtr->nWidth; - if (delta > infoPtr->nButtonSize) - infoPtr->nWidth += 4 * infoPtr->nButtonSize / 3; - else if (delta > 0) - infoPtr->nWidth += infoPtr->nButtonSize / 3; - } - - h = wndRect.bottom - wndRect.top + infoPtr->nButtonSize; - - TRACE("[%p] infoPtr->nWidth set to %d\n", - infoPtr->hwndSelf, infoPtr->nWidth); - - return h; -} - -static INT -PAGER_SetFixedHeight(PAGER_INFO* infoPtr) -{ - /* Must set the non-scrollable dimension to be less than the full height/width - * so that NCCalcSize is called. The Microsoft docs mention 3/4 factor for button - * size, and experimentation shows that the effect is almost right. */ - - RECT wndRect; - INT delta, w; - GetWindowRect(infoPtr->hwndSelf, &wndRect); - - /* see what the app says for btn height */ - PAGER_CalcSize(infoPtr, &infoPtr->nHeight, FALSE); - - if (infoPtr->dwStyle & CCS_NORESIZE) - { - delta = wndRect.bottom - wndRect.top - infoPtr->nHeight; - if (delta > infoPtr->nButtonSize) - infoPtr->nHeight += infoPtr->nButtonSize; - else if (delta > 0) - infoPtr->nHeight += infoPtr->nButtonSize / 3; - } - - w = wndRect.right - wndRect.left + infoPtr->nButtonSize; - - TRACE("[%p] infoPtr->nHeight set to %d\n", - infoPtr->hwndSelf, infoPtr->nHeight); - - return w; -} - /****************************************************************** * For the PGM_RECALCSIZE message (but not the other uses in * * this module), the native control does only the following: * @@ -693,28 +500,14 @@ static LRESULT PAGER_SetChild (PAGER_INFO* infoPtr, HWND hwndChild) { - INT hw; - infoPtr->hwndChild = IsWindow (hwndChild) ? hwndChild : 0; if (infoPtr->hwndChild) { TRACE("[%p] hwndChild=%p\n", infoPtr->hwndSelf, infoPtr->hwndChild); - if (infoPtr->dwStyle & PGS_HORZ) { - hw = PAGER_SetFixedHeight(infoPtr); - /* adjust non-scrollable dimension to fit the child */ - SetWindowPos(infoPtr->hwndSelf, 0, 0, 0, hw, infoPtr->nHeight, - SWP_FRAMECHANGED | SWP_NOMOVE | SWP_NOZORDER | - SWP_NOSIZE | SWP_NOACTIVATE); - } - else { - hw = PAGER_SetFixedWidth(infoPtr); - /* adjust non-scrollable dimension to fit the child */ - SetWindowPos(infoPtr->hwndSelf, 0, 0, 0, infoPtr->nWidth, hw, - SWP_FRAMECHANGED | SWP_NOMOVE | SWP_NOZORDER | - SWP_NOSIZE | SWP_NOACTIVATE); - } + SetWindowPos(infoPtr->hwndSelf, 0, 0, 0, 0, 0, + SWP_FRAMECHANGED | SWP_NOMOVE | SWP_NOZORDER | SWP_NOSIZE | SWP_NOACTIVATE); /* position child within the page scroller */ SetWindowPos(infoPtr->hwndChild, HWND_TOP, @@ -853,11 +646,12 @@ MapWindowPoints (0, infoPtr->hwndSelf, (LPPOINT)&rcChild, 2); /* FIXME: RECT != 2 POINTS */ GetWindowRect (infoPtr->hwndSelf, &rcWindow); + infoPtr->nWidth = lpRect->right - lpRect->left; + infoPtr->nHeight = lpRect->bottom - lpRect->top; + PAGER_CalcSize( infoPtr ); + if (infoPtr->dwStyle & PGS_HORZ) { - infoPtr->nWidth = lpRect->right - lpRect->left; - PAGER_CalcSize (infoPtr, &infoPtr->nWidth, TRUE); - if (infoPtr->TLbtnState && (lpRect->left + infoPtr->nButtonSize < lpRect->right)) lpRect->left += infoPtr->nButtonSize; if (infoPtr->BRbtnState && (lpRect->right - infoPtr->nButtonSize > lpRect->left)) @@ -865,9 +659,6 @@ } else { - infoPtr->nHeight = lpRect->bottom - lpRect->top; - PAGER_CalcSize (infoPtr, &infoPtr->nHeight, FALSE); - if (infoPtr->TLbtnState && (lpRect->top + infoPtr->nButtonSize < lpRect->bottom)) lpRect->top += infoPtr->nButtonSize; if (infoPtr->BRbtnState && (lpRect->bottom - infoPtr->nButtonSize > lpRect->top)) Modified: trunk/reactos/dll/win32/comctl32/progress.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/progres… ============================================================================== --- trunk/reactos/dll/win32/comctl32/progress.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/progress.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -745,7 +745,7 @@ ZeroMemory (&wndClass, sizeof(wndClass)); wndClass.style = CS_GLOBALCLASS | CS_VREDRAW | CS_HREDRAW; - wndClass.lpfnWndProc = (WNDPROC)ProgressWindowProc; + wndClass.lpfnWndProc = ProgressWindowProc; wndClass.cbClsExtra = 0; wndClass.cbWndExtra = sizeof (PROGRESS_INFO *); wndClass.hCursor = LoadCursorW (0, (LPWSTR)IDC_ARROW); Modified: trunk/reactos/dll/win32/comctl32/propsheet.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/propshe… ============================================================================== --- trunk/reactos/dll/win32/comctl32/propsheet.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/propsheet.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -171,6 +171,7 @@ static int PROPSHEET_GetPageIndex(HPROPSHEETPAGE hpage, const PropSheetInfo* psInfo); static PADDING_INFO PROPSHEET_GetPaddingInfoWizard(HWND hwndDlg, const PropSheetInfo* psInfo); static BOOL PROPSHEET_DoCommand(HWND hwnd, WORD wID); +static BOOL PROPSHEET_RemovePage(HWND hwndDlg, int index, HPROPSHEETPAGE hpage); static INT_PTR CALLBACK PROPSHEET_DialogProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam); @@ -554,13 +555,12 @@ if (IS_INTRESOURCE( lppsp->pszTitle )) { - if (!LoadStringW( lppsp->hInstance, (DWORD_PTR)lppsp->pszTitle,szTitle,sizeof(szTitle)/sizeof(szTitle[0]) )) - { + if (LoadStringW( lppsp->hInstance, (DWORD_PTR)lppsp->pszTitle, szTitle, sizeof(szTitle)/sizeof(szTitle[0]) )) + pTitle = szTitle; + else if (*p) + pTitle = p; + else pTitle = pszNull; - FIXME("Could not load resource #%04x?\n",LOWORD(lppsp->pszTitle)); - } - else - pTitle = szTitle; } else pTitle = lppsp->pszTitle; @@ -1173,7 +1173,7 @@ SendMessageW(hwndTabCtrl, TCM_SETIMAGELIST, 0, (LPARAM)psInfo->hImageList); } - SendMessageW(GetDlgItem(hwndTabCtrl, IDC_TABCONTROL), WM_SETREDRAW, 0, 0); + SendMessageW(hwndTabCtrl, WM_SETREDRAW, 0, 0); for (i = 0; i < nTabs; i++) { if ( psInfo->proppage[i].hasIcon ) @@ -1189,7 +1189,7 @@ item.pszText = (LPWSTR) psInfo->proppage[i].pszText; SendMessageW(hwndTabCtrl, TCM_INSERTITEMW, i, (LPARAM)&item); } - SendMessageW(GetDlgItem(hwndTabCtrl, IDC_TABCONTROL), WM_SETREDRAW, 1, 0); + SendMessageW(hwndTabCtrl, WM_SETREDRAW, 1, 0); return TRUE; } @@ -1473,6 +1473,9 @@ (LPARAM)ppshpage); /* Free a no more needed copy */ Free(pTemplateCopy); + + if(!hwndPage) + return FALSE; psInfo->proppage[index].hwndPage = hwndPage; @@ -2025,7 +2028,14 @@ psn.lParam = 0; if (!psInfo->proppage[index].hwndPage) { - PROPSHEET_CreatePage(hwndDlg, index, psInfo, ppshpage); + if(!PROPSHEET_CreatePage(hwndDlg, index, psInfo, ppshpage)) { + PROPSHEET_RemovePage(hwndDlg, index, NULL); + if(index >= psInfo->nPages) + index--; + if(index < 0) + return FALSE; + continue; + } } /* Resize the property sheet page to the fit in the Tab control @@ -2273,7 +2283,8 @@ if (ppsp->dwFlags & PSP_PREMATURE) { /* Create the page but don't show it */ - PROPSHEET_CreatePage(hwndDlg, psInfo->nPages, psInfo, ppsp); + if(!PROPSHEET_CreatePage(hwndDlg, psInfo->nPages, psInfo, ppsp)) + return FALSE; } /* Modified: trunk/reactos/dll/win32/comctl32/rebar.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/rebar.c… ============================================================================== --- trunk/reactos/dll/win32/comctl32/rebar.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/rebar.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -2135,8 +2135,10 @@ if(yOff < 0) { /* Place the band above the current top row */ + if(iHitBand==0 && (infoPtr->uNumBands==1 || REBAR_GetBand(infoPtr, 1)->fStyle&RBBS_BREAK)) + return; DPA_DeletePtr(infoPtr->bands, iHitBand); - hitBand->fStyle &= RBBS_BREAK; + hitBand->fStyle &= ~RBBS_BREAK; REBAR_GetBand(infoPtr, 0)->fStyle |= RBBS_BREAK; infoPtr->iGrabbedBand = DPA_InsertPtr( infoPtr->bands, 0, hitBand); @@ -2144,6 +2146,8 @@ else if(yOff > REBAR_GetBand(infoPtr, infoPtr->uNumBands - 1)->rcBand.bottom) { /* Place the band below the current bottom row */ + if(iHitBand == infoPtr->uNumBands-1 && hitBand->fStyle&RBBS_BREAK) + return; DPA_DeletePtr(infoPtr->bands, iHitBand); hitBand->fStyle |= RBBS_BREAK; infoPtr->iGrabbedBand = DPA_InsertPtr( Modified: trunk/reactos/dll/win32/comctl32/status.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/status.… ============================================================================== --- trunk/reactos/dll/win32/comctl32/status.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/status.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -1284,7 +1284,7 @@ return STATUSBAR_WMGetText (infoPtr, (INT)wParam, (LPWSTR)lParam); case WM_GETTEXTLENGTH: - return STATUSBAR_GetTextLength (infoPtr, 0); + return LOWORD(STATUSBAR_GetTextLength (infoPtr, 0)); case WM_LBUTTONDBLCLK: return STATUSBAR_SendMouseNotify(infoPtr, NM_DBLCLK, msg, wParam, lParam); Modified: trunk/reactos/dll/win32/comctl32/string.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/string.… ============================================================================== --- trunk/reactos/dll/win32/comctl32/string.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/string.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -77,7 +77,7 @@ else str2[1] = '\0'; - return CompareStringA(GetThreadLocale(), dwFlags, str1, -1, str2, -1) - 2; + return CompareStringA(GetThreadLocale(), dwFlags, str1, -1, str2, -1) - CSTR_EQUAL; } /************************************************************************* @@ -117,7 +117,7 @@ */ static inline BOOL COMCTL32_ChrCmpIW(WCHAR ch1, WCHAR ch2) { - return CompareStringW(GetThreadLocale(), NORM_IGNORECASE, &ch1, 1, &ch2, 1) - 2; + return CompareStringW(GetThreadLocale(), NORM_IGNORECASE, &ch1, 1, &ch2, 1) - CSTR_EQUAL; } /************************************************************************** @@ -299,12 +299,8 @@ */ INT WINAPI StrCmpNIA(LPCSTR lpszStr, LPCSTR lpszComp, INT iLen) { - INT iRet; - TRACE("(%s,%s,%i)\n", debugstr_a(lpszStr), debugstr_a(lpszComp), iLen); - - iRet = CompareStringA(GetThreadLocale(), NORM_IGNORECASE, lpszStr, iLen, lpszComp, iLen); - return iRet == CSTR_LESS_THAN ? -1 : iRet == CSTR_GREATER_THAN ? 1 : 0; + return CompareStringA(GetThreadLocale(), NORM_IGNORECASE, lpszStr, iLen, lpszComp, iLen) - CSTR_EQUAL; } /************************************************************************* @@ -314,12 +310,8 @@ */ INT WINAPI StrCmpNIW(LPCWSTR lpszStr, LPCWSTR lpszComp, INT iLen) { - INT iRet; - TRACE("(%s,%s,%i)\n", debugstr_w(lpszStr), debugstr_w(lpszComp), iLen); - - iRet = CompareStringW(GetThreadLocale(), NORM_IGNORECASE, lpszStr, iLen, lpszComp, iLen); - return iRet == CSTR_LESS_THAN ? -1 : iRet == CSTR_GREATER_THAN ? 1 : 0; + return CompareStringW(GetThreadLocale(), NORM_IGNORECASE, lpszStr, iLen, lpszComp, iLen) - CSTR_EQUAL; } /************************************************************************* @@ -496,12 +488,8 @@ */ INT WINAPI StrCmpNA(LPCSTR lpszStr, LPCSTR lpszComp, INT iLen) { - INT iRet; - TRACE("(%s,%s,%i)\n", debugstr_a(lpszStr), debugstr_a(lpszComp), iLen); - - iRet = CompareStringA(GetThreadLocale(), 0, lpszStr, iLen, lpszComp, iLen); - return iRet == CSTR_LESS_THAN ? -1 : iRet == CSTR_GREATER_THAN ? 1 : 0; + return CompareStringA(GetThreadLocale(), 0, lpszStr, iLen, lpszComp, iLen) - CSTR_EQUAL; } /************************************************************************** @@ -511,12 +499,8 @@ */ INT WINAPI StrCmpNW(LPCWSTR lpszStr, LPCWSTR lpszComp, INT iLen) { - INT iRet; - TRACE("(%s,%s,%i)\n", debugstr_w(lpszStr), debugstr_w(lpszComp), iLen); - - iRet = CompareStringW(GetThreadLocale(), 0, lpszStr, iLen, lpszComp, iLen); - return iRet == CSTR_LESS_THAN ? -1 : iRet == CSTR_GREATER_THAN ? 1 : 0; + return CompareStringW(GetThreadLocale(), 0, lpszStr, iLen, lpszComp, iLen) - CSTR_EQUAL; } /************************************************************************** @@ -877,10 +861,11 @@ TRACE("(%d,%s,%s,%d)\n", bCase, debugstr_a(lpszStr), debugstr_a(lpszComp), iLen); - /* FIXME: These flags are undocumented and unknown by our CompareString. - * We need defines for them. + /* FIXME: This flag is undocumented and unknown by our CompareString. + * We need a define for it. */ - dwFlags |= bCase ? 0x10000000 : 0x10000001; + dwFlags = 0x10000000; + if (!bCase) dwFlags |= NORM_IGNORECASE; iRet = CompareStringA(GetThreadLocale(), dwFlags, lpszStr, iLen, lpszComp, iLen); @@ -888,7 +873,7 @@ if (!iRet) iRet = CompareStringA(2048, dwFlags, lpszStr, iLen, lpszComp, iLen); - return iRet == 2 ? TRUE : FALSE; + return iRet == CSTR_EQUAL; } /************************************************************************* @@ -905,10 +890,11 @@ TRACE("(%d,%s,%s,%d)\n", bCase, debugstr_w(lpszStr),debugstr_w(lpszComp), iLen); - /* FIXME: These flags are undocumented and unknown by our CompareString. - * We need defines for them. + /* FIXME: This flag is undocumented and unknown by our CompareString. + * We need a define for it. */ - dwFlags = bCase ? 0x10000000 : 0x10000001; + dwFlags = 0x10000000; + if (!bCase) dwFlags |= NORM_IGNORECASE; iRet = CompareStringW(GetThreadLocale(), dwFlags, lpszStr, iLen, lpszComp, iLen); @@ -916,5 +902,5 @@ if (!iRet) iRet = CompareStringW(2048, dwFlags, lpszStr, iLen, lpszComp, iLen); - return iRet == 2 ? TRUE : FALSE; -} + return iRet == CSTR_EQUAL; +} Modified: trunk/reactos/dll/win32/comctl32/tab.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/tab.c?r… ============================================================================== --- trunk/reactos/dll/win32/comctl32/tab.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/tab.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -2879,13 +2879,12 @@ if (iItem < 0 || iItem >= infoPtr->uNumItem) return FALSE; + TAB_InvalidateTabArea(infoPtr); item = TAB_GetItem(infoPtr, iItem); Free(item->pszText); Free(item); infoPtr->uNumItem--; DPA_DeletePtr(infoPtr->items, iItem); - - TAB_InvalidateTabArea(infoPtr); if (infoPtr->uNumItem == 0) { @@ -3021,7 +3020,7 @@ TEXTMETRICW fontMetrics; HDC hdc; HFONT hOldFont; - DWORD dwStyle; + DWORD style; infoPtr = Alloc (sizeof(TAB_INFO)); @@ -3055,11 +3054,13 @@ /* The tab control always has the WS_CLIPSIBLINGS style. Even if you don't specify it in CreateWindow. This is necessary in order for paint to work correctly. This follows windows behaviour. */ - dwStyle = GetWindowLongW(hwnd, GWL_STYLE); - SetWindowLongW(hwnd, GWL_STYLE, dwStyle|WS_CLIPSIBLINGS); - - infoPtr->dwStyle = dwStyle | WS_CLIPSIBLINGS; - infoPtr->exStyle = (dwStyle & TCS_FLATBUTTONS) ? TCS_EX_FLATSEPARATORS : 0; + style = GetWindowLongW(hwnd, GWL_STYLE); + if (style & TCS_VERTICAL) style |= TCS_MULTILINE; + style |= WS_CLIPSIBLINGS; + SetWindowLongW(hwnd, GWL_STYLE, style); + + infoPtr->dwStyle = style; + infoPtr->exStyle = (style & TCS_FLATBUTTONS) ? TCS_EX_FLATSEPARATORS : 0; if (infoPtr->dwStyle & TCS_TOOLTIPS) { /* Create tooltip control */ Modified: trunk/reactos/dll/win32/comctl32/toolbar.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/toolbar… ============================================================================== --- trunk/reactos/dll/win32/comctl32/toolbar.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/toolbar.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -217,9 +217,6 @@ #define TOOLBAR_NOWHERE (-1) -#define TOOLBAR_HasText(x, y) (TOOLBAR_GetText(x, y) ? TRUE : FALSE) -#define TOOLBAR_HasDropDownArrows(exStyle) ((exStyle & TBSTYLE_EX_DRAWDDARROWS) ? TRUE : FALSE) - /* Used to find undocumented extended styles */ #define TBSTYLE_EX_ALL (TBSTYLE_EX_DRAWDDARROWS | \ TBSTYLE_EX_UNDOC1 | \ @@ -258,6 +255,11 @@ return (infoPtr->dwStyle & TBSTYLE_FLAT ? 0 : TOP_BORDER); } +static inline BOOL TOOLBAR_HasDropDownArrows(DWORD exStyle) +{ + return (exStyle & TBSTYLE_EX_DRAWDDARROWS) != 0; +} + static LPWSTR TOOLBAR_GetText(const TOOLBAR_INFO *infoPtr, const TBUTTON_INFO *btnPtr) { @@ -428,7 +430,7 @@ if (!TOOLBAR_IsValidBitmapIndex(infoPtr,btnPtr->iBitmap)) { if (btnPtr->iBitmap == I_IMAGENONE) return NULL; - ERR("bitmap for ID %d, index %d is not valid, number of bitmaps in imagelist: %d\n", + WARN("bitmap for ID %d, index %d is not valid, number of bitmaps in imagelist: %d\n", HIWORD(btnPtr->iBitmap), LOWORD(btnPtr->iBitmap), infoPtr->nNumBitmaps); return NULL; } @@ -1247,7 +1249,7 @@ btnPtr = infoPtr->buttons; for (i = 0; i < infoPtr->nNumButtons; i++, btnPtr++) { - if(TOOLBAR_HasText(infoPtr, btnPtr)) + if(TOOLBAR_GetText(infoPtr, btnPtr)) { TOOLBAR_MeasureString(infoPtr, btnPtr, hdc, &sz); if (sz.cx > lpSize->cx) @@ -1775,10 +1777,13 @@ static INT -TOOLBAR_InternalHitTest (const TOOLBAR_INFO *infoPtr, const POINT *lpPt) +TOOLBAR_InternalHitTest (const TOOLBAR_INFO *infoPtr, const POINT *lpPt, BOOL *button) { TBUTTON_INFO *btnPtr; INT i; + + if (button) + *button = FALSE; btnPtr = infoPtr->buttons; for (i = 0; i < infoPtr->nNumButtons; i++, btnPtr++) { @@ -1794,6 +1799,8 @@ else { if (PtInRect (&btnPtr->rect, *lpPt)) { TRACE(" ON BUTTON %d!\n", i); + if (button) + *button = TRUE; return i; } } @@ -2399,7 +2406,7 @@ nmtb.tbButton.iString >= 0 ? debugstr_w(infoPtr->strings[nmtb.tbButton.iString]) : ""); - /* insert button into the apropriate list */ + /* insert button into the appropriate list */ index = TOOLBAR_GetButtonIndex (custInfo->tbInfo, nmtb.tbButton.idCommand, FALSE); if (index == -1) { @@ -3133,7 +3140,7 @@ btnPtr = &infoPtr->buttons[nIndex]; - bChecked = (btnPtr->fsState & TBSTATE_CHECKED) ? TRUE : FALSE; + bChecked = (btnPtr->fsState & TBSTATE_CHECKED) != 0; if (LOWORD(lParam) == FALSE) btnPtr->fsState &= ~TBSTATE_CHECKED; @@ -3664,7 +3671,7 @@ static inline LRESULT TOOLBAR_HitTest (const TOOLBAR_INFO *infoPtr, const POINT* lpPt) { - return TOOLBAR_InternalHitTest (infoPtr, lpPt); + return TOOLBAR_InternalHitTest (infoPtr, lpPt, NULL); } @@ -4481,20 +4488,22 @@ * (MSDN says that this parameter is reserved) */ static LRESULT -TOOLBAR_SetExtendedStyle (TOOLBAR_INFO *infoPtr, LPARAM lParam) -{ - DWORD dwOldStyle; - - dwOldStyle = infoPtr->dwExStyle; - infoPtr->dwExStyle = (DWORD)lParam; - - TRACE("new style 0x%08x\n", infoPtr->dwExStyle); +TOOLBAR_SetExtendedStyle (TOOLBAR_INFO *infoPtr, DWORD mask, DWORD style) +{ + DWORD old_style = infoPtr->dwExStyle; + + TRACE("mask=0x%08x, style=0x%08x\n", mask, style); + + if (mask) + infoPtr->dwExStyle = (old_style & ~mask) | (style & mask); + else + infoPtr->dwExStyle = style; if (infoPtr->dwExStyle & ~TBSTYLE_EX_ALL) FIXME("Unknown Toolbar Extended Style 0x%08x. Please report.\n", (infoPtr->dwExStyle & ~TBSTYLE_EX_ALL)); - if ((dwOldStyle ^ infoPtr->dwExStyle) & TBSTYLE_EX_MIXEDBUTTONS) + if ((old_style ^ infoPtr->dwExStyle) & TBSTYLE_EX_MIXEDBUTTONS) TOOLBAR_CalcToolbar(infoPtr); else TOOLBAR_LayoutToolbar(infoPtr); @@ -4502,7 +4511,7 @@ TOOLBAR_AutoSize(infoPtr); InvalidateRect(infoPtr->hwndSelf, NULL, TRUE); - return (LRESULT)dwOldStyle; + return old_style; } @@ -5155,7 +5164,7 @@ TRACE("hwnd = %p, style=0x%08x\n", hwnd, lpcs->style); - infoPtr->dwStyle = lpcs->style; + infoPtr->dwStyle = GetWindowLongW(hwnd, GWL_STYLE); GetClientRect(hwnd, &infoPtr->client_rect); infoPtr->bUnicode = infoPtr->hwndNotify && (NFR_UNICODE == SendMessageW(hwnd, WM_NOTIFYFORMAT, (WPARAM)hwnd, NF_REQUERY)); @@ -5374,13 +5383,13 @@ TOOLBAR_LButtonDblClk (TOOLBAR_INFO *infoPtr, WPARAM wParam, LPARAM lParam) { POINT pt; - INT nHit; + BOOL button; pt.x = (short)LOWORD(lParam); pt.y = (short)HIWORD(lParam); - nHit = TOOLBAR_InternalHitTest (infoPtr, &pt); - - if (nHit >= 0) + TOOLBAR_InternalHitTest (infoPtr, &pt, &button); + + if (button) TOOLBAR_LButtonDown (infoPtr, wParam, lParam); else if (infoPtr->dwStyle & CCS_ADJUSTABLE) TOOLBAR_Customize (infoPtr); @@ -5398,6 +5407,7 @@ NMTOOLBARA nmtb; NMMOUSE nmmouse; BOOL bDragKeyPressed; + BOOL button; TRACE("\n"); @@ -5412,11 +5422,12 @@ pt.x = (short)LOWORD(lParam); pt.y = (short)HIWORD(lParam); - nHit = TOOLBAR_InternalHitTest (infoPtr, &pt); - - btnPtr = &infoPtr->buttons[nHit]; - - if ((nHit >= 0) && bDragKeyPressed && (infoPtr->dwStyle & CCS_ADJUSTABLE)) + nHit = TOOLBAR_InternalHitTest (infoPtr, &pt, &button); + + if (button) + btnPtr = &infoPtr->buttons[nHit]; + + if (button && bDragKeyPressed && (infoPtr->dwStyle & CCS_ADJUSTABLE)) { infoPtr->nButtonDrag = nHit; SetCapture (infoPtr->hwndSelf); @@ -5427,7 +5438,7 @@ hCursorDrag = LoadCursorW(COMCTL32_hModule, (LPCWSTR)IDC_MOVEBUTTON); SetCursor(hCursorDrag); } - else if (nHit >= 0) + else if (button) { RECT arrowRect; infoPtr->nOldHit = nHit; @@ -5474,9 +5485,9 @@ * NOTE: native doesn't do this, but that is a bug */ GetCursorPos(&pt); ScreenToClient(infoPtr->hwndSelf, &pt); - nHit = TOOLBAR_InternalHitTest(infoPtr, &pt); - if (!infoPtr->bAnchor || (nHit >= 0)) - TOOLBAR_SetHotItemEx(infoPtr, nHit, HICF_MOUSE | HICF_LMOUSE); + nHit = TOOLBAR_InternalHitTest(infoPtr, &pt, &button); + if (!infoPtr->bAnchor || button) + TOOLBAR_SetHotItemEx(infoPtr, button ? nHit : TOOLBAR_NOWHERE, HICF_MOUSE | HICF_LMOUSE); /* remove any left mouse button down or double-click messages * so that we can get a toggle effect on the button */ @@ -5494,7 +5505,7 @@ btnPtr->fsState |= TBSTATE_PRESSED; - TOOLBAR_SetHotItemEx(infoPtr, nHit, HICF_MOUSE | HICF_LMOUSE); + TOOLBAR_SetHotItemEx(infoPtr, button ? nHit : TOOLBAR_NOWHERE, HICF_MOUSE | HICF_LMOUSE); if (btnPtr->fsState & TBSTATE_ENABLED) InvalidateRect(infoPtr->hwndSelf, &btnPtr->rect, TRUE); @@ -5502,7 +5513,7 @@ SetCapture (infoPtr->hwndSelf); } - if (nHit >=0) + if (button) { memset(&nmtb, 0, sizeof(nmtb)); nmtb.iItem = btnPtr->idCommand; @@ -5512,7 +5523,7 @@ nmmouse.dwHitInfo = nHit; /* !!! Undocumented - sends NM_LDOWN with the NMMOUSE structure. */ - if (nHit < 0) + if (!button) nmmouse.dwItemSpec = -1; else { @@ -5539,6 +5550,7 @@ NMHDR hdr; NMMOUSE nmmouse; NMTOOLBARA nmtb; + BOOL button; if (infoPtr->hwndToolTip) TOOLBAR_RelayEvent (infoPtr->hwndToolTip, infoPtr->hwndSelf, @@ -5546,10 +5558,10 @@ pt.x = (short)LOWORD(lParam); pt.y = (short)HIWORD(lParam); - nHit = TOOLBAR_InternalHitTest (infoPtr, &pt); - - if (!infoPtr->bAnchor || (nHit >= 0)) - TOOLBAR_SetHotItemEx(infoPtr, nHit, HICF_MOUSE | HICF_LMOUSE); + nHit = TOOLBAR_InternalHitTest (infoPtr, &pt, &button); + + if (!infoPtr->bAnchor || button) + TOOLBAR_SetHotItemEx(infoPtr, button ? nHit : TOOLBAR_NOWHERE, HICF_MOUSE | HICF_LMOUSE); if (infoPtr->nButtonDrag >= 0) { RECT rcClient; @@ -5609,7 +5621,7 @@ } /* button under cursor changed so need to re-set hot item */ - TOOLBAR_SetHotItemEx(infoPtr, nHit, HICF_MOUSE | HICF_LMOUSE); + TOOLBAR_SetHotItemEx(infoPtr, button ? nHit : TOOLBAR_NOWHERE, HICF_MOUSE | HICF_LMOUSE); infoPtr->nButtonDrag = -1; TOOLBAR_SendNotify(&hdr, infoPtr, TBN_TOOLBARCHANGE); @@ -5674,7 +5686,7 @@ * NM_CLICK with the NMMOUSE structure. */ nmmouse.dwHitInfo = nHit; - if (nHit < 0) + if (!button) nmmouse.dwItemSpec = -1; else { @@ -5697,14 +5709,15 @@ INT nHit; NMMOUSE nmmouse; POINT pt; + BOOL button; pt.x = (short)LOWORD(lParam); pt.y = (short)HIWORD(lParam); - nHit = TOOLBAR_InternalHitTest(infoPtr, &pt); + nHit = TOOLBAR_InternalHitTest(infoPtr, &pt, &button); nmmouse.dwHitInfo = nHit; - if (nHit < 0) { + if (!button) { nmmouse.dwItemSpec = -1; } else { nmmouse.dwItemSpec = infoPtr->buttons[nmmouse.dwHitInfo].idCommand; @@ -5803,6 +5816,7 @@ TRACKMOUSEEVENT trackinfo; INT nHit; TBUTTON_INFO *btnPtr; + BOOL button; if ((infoPtr->dwStyle & TBSTYLE_TOOLTIPS) && (infoPtr->hwndToolTip == NULL)) TOOLBAR_TooltipCreateControl(infoPtr); @@ -5833,11 +5847,11 @@ pt.x = (short)LOWORD(lParam); pt.y = (short)HIWORD(lParam); - nHit = TOOLBAR_InternalHitTest (infoPtr, &pt); + nHit = TOOLBAR_InternalHitTest (infoPtr, &pt, &button); if (((infoPtr->dwStyle & TBSTYLE_FLAT) || GetWindowTheme (infoPtr->hwndSelf)) - && (!infoPtr->bAnchor || (nHit >= 0))) - TOOLBAR_SetHotItemEx(infoPtr, nHit, HICF_MOUSE); + && (!infoPtr->bAnchor || button)) + TOOLBAR_SetHotItemEx(infoPtr, button ? nHit : TOOLBAR_NOWHERE, HICF_MOUSE); if (infoPtr->nOldHit != nHit) { @@ -6629,7 +6643,7 @@ return TOOLBAR_SetDrawTextFlags (infoPtr, wParam, lParam); case TB_SETEXTENDEDSTYLE: - return TOOLBAR_SetExtendedStyle (infoPtr, lParam); + return TOOLBAR_SetExtendedStyle (infoPtr, wParam, lParam); case TB_SETHOTIMAGELIST: return TOOLBAR_SetHotImageList (infoPtr, wParam, (HIMAGELIST)lParam); Modified: trunk/reactos/dll/win32/comctl32/tooltips.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/tooltip… ============================================================================== --- trunk/reactos/dll/win32/comctl32/tooltips.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/tooltips.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -171,6 +171,8 @@ #define ICON_HEIGHT 16 #define ICON_WIDTH 16 +#define MAX_TEXT_SIZE_A 80 /* maximum retrieving text size by ANSI message */ + static LRESULT CALLBACK TOOLTIPS_SubclassProc (HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam, UINT_PTR uId, DWORD_PTR dwRef); @@ -943,6 +945,22 @@ return -1; } +static inline void +TOOLTIPS_CopyInfoT (const TTTOOL_INFO *toolPtr, TTTOOLINFOW *ti, BOOL isW) +{ + if (ti->lpszText) { + if (toolPtr->lpszText == NULL || + IS_INTRESOURCE(toolPtr->lpszText) || + toolPtr->lpszText == LPSTR_TEXTCALLBACKW) + ti->lpszText = toolPtr->lpszText; + else if (isW) + strcpyW (ti->lpszText, toolPtr->lpszText); + else + /* ANSI version, the buffer is maximum 80 bytes without null. */ + WideCharToMultiByte(CP_ACP, 0, toolPtr->lpszText, -1, + (LPSTR)ti->lpszText, MAX_TEXT_SIZE_A, NULL, NULL); + } +} static BOOL TOOLTIPS_IsWindowActive (HWND hwnd) @@ -1199,8 +1217,7 @@ ti->uId = toolPtr->uId; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* ti->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOA_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1246,8 +1263,7 @@ ti->uFlags = toolPtr->uFlags; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* ti->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOW_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1328,12 +1344,13 @@ /* NB this API is broken, there is no way for the app to determine what size buffer it requires nor a way to specify how long the - one it supplies is. We'll assume it's up to INFOTIPSIZE */ + one it supplies is. According to the test result, it's up to + 80 bytes by the ANSI version. */ buffer[0] = '\0'; TOOLTIPS_GetTipText(infoPtr, nTool, buffer); WideCharToMultiByte(CP_ACP, 0, buffer, -1, (LPSTR)ti->lpszText, - INFOTIPSIZE, NULL, NULL); + MAX_TEXT_SIZE_A, NULL, NULL); } return 0; @@ -1385,8 +1402,7 @@ ti->uFlags = toolPtr->uFlags; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* lpToolInfo->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOW_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1420,8 +1436,7 @@ lptthit->ti.uId = toolPtr->uId; lptthit->ti.rect = toolPtr->rect; lptthit->ti.hinst = toolPtr->hinst; -/* lptthit->ti.lpszText = toolPtr->lpszText; */ - lptthit->ti.lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, &lptthit->ti, isW); if (lptthit->ti.cbSize >= TTTOOLINFOW_V2_SIZE) lptthit->ti.lParam = toolPtr->lParam; } Modified: trunk/reactos/dll/win32/comctl32/trackbar.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/trackba… ============================================================================== --- trunk/reactos/dll/win32/comctl32/trackbar.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/trackbar.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -457,7 +457,7 @@ TRACKBAR_PageUp(infoPtr); else return FALSE; - infoPtr->flags |= TB_THUMBPOSCHANGED; + TRACKBAR_UpdateThumb (infoPtr); TRACKBAR_InvalidateThumbMove (infoPtr, prevPos, infoPtr->lPos); return TRUE; @@ -1151,10 +1151,13 @@ static inline LRESULT -TRACKBAR_SetRange (TRACKBAR_INFO *infoPtr, BOOL fRedraw, LONG lRange) -{ - infoPtr->lRangeMin = (SHORT)LOWORD(lRange); - infoPtr->lRangeMax = (SHORT)HIWORD(lRange); +TRACKBAR_SetRange (TRACKBAR_INFO *infoPtr, BOOL redraw, LONG range) +{ + BOOL changed = infoPtr->lRangeMin != (SHORT)LOWORD(range) || + infoPtr->lRangeMax != (SHORT)HIWORD(range); + + infoPtr->lRangeMin = (SHORT)LOWORD(range); + infoPtr->lRangeMax = (SHORT)HIWORD(range); if (infoPtr->lPos < infoPtr->lRangeMin) { infoPtr->lPos = infoPtr->lRangeMin; @@ -1169,15 +1172,20 @@ infoPtr->lPageSize = (infoPtr->lRangeMax - infoPtr->lRangeMin) / 5; if (infoPtr->lPageSize == 0) infoPtr->lPageSize = 1; - if (fRedraw) TRACKBAR_InvalidateAll(infoPtr); + if (changed && (infoPtr->dwStyle & TBS_AUTOTICKS)) + TRACKBAR_RecalculateTics (infoPtr); + + if (redraw) TRACKBAR_InvalidateAll(infoPtr); return 0; } static inline LRESULT -TRACKBAR_SetRangeMax (TRACKBAR_INFO *infoPtr, BOOL fRedraw, LONG lMax) -{ +TRACKBAR_SetRangeMax (TRACKBAR_INFO *infoPtr, BOOL redraw, LONG lMax) +{ + BOOL changed = infoPtr->lRangeMax != lMax; + infoPtr->lRangeMax = lMax; if (infoPtr->lPos > infoPtr->lRangeMax) { infoPtr->lPos = infoPtr->lRangeMax; @@ -1187,15 +1195,20 @@ infoPtr->lPageSize = (infoPtr->lRangeMax - infoPtr->lRangeMin) / 5; if (infoPtr->lPageSize == 0) infoPtr->lPageSize = 1; - if (fRedraw) TRACKBAR_InvalidateAll(infoPtr); + if (changed && (infoPtr->dwStyle & TBS_AUTOTICKS)) + TRACKBAR_RecalculateTics (infoPtr); + + if (redraw) TRACKBAR_InvalidateAll(infoPtr); return 0; } static inline LRESULT -TRACKBAR_SetRangeMin (TRACKBAR_INFO *infoPtr, BOOL fRedraw, LONG lMin) -{ +TRACKBAR_SetRangeMin (TRACKBAR_INFO *infoPtr, BOOL redraw, LONG lMin) +{ + BOOL changed = infoPtr->lRangeMin != lMin; + infoPtr->lRangeMin = lMin; if (infoPtr->lPos < infoPtr->lRangeMin) { infoPtr->lPos = infoPtr->lRangeMin; @@ -1205,7 +1218,10 @@ infoPtr->lPageSize = (infoPtr->lRangeMax - infoPtr->lRangeMin) / 5; if (infoPtr->lPageSize == 0) infoPtr->lPageSize = 1; - if (fRedraw) TRACKBAR_InvalidateAll(infoPtr); + if (changed && (infoPtr->dwStyle & TBS_AUTOTICKS)) + TRACKBAR_RecalculateTics (infoPtr); + + if (redraw) TRACKBAR_InvalidateAll(infoPtr); return 0; } @@ -1648,10 +1664,9 @@ if (dragPos == oldPos) return TRUE; infoPtr->lPos = dragPos; - - infoPtr->flags |= TB_THUMBPOSCHANGED; + TRACKBAR_UpdateThumb (infoPtr); + notify_with_scroll (infoPtr, TB_THUMBTRACK | (infoPtr->lPos<<16)); - TRACKBAR_InvalidateThumbMove(infoPtr, oldPos, dragPos); UpdateWindow (infoPtr->hwndSelf); Modified: trunk/reactos/dll/win32/comctl32/treeview.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/comctl32/treevie… ============================================================================== --- trunk/reactos/dll/win32/comctl32/treeview.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/comctl32/treeview.c [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -66,13 +66,72 @@ WINE_DEFAULT_DEBUG_CHANNEL(treeview); -enum StateListType -{ - OriginInternal, - OriginUser -}; - /* internal structures */ +typedef struct tagTREEVIEW_INFO +{ + HWND hwnd; + HWND hwndNotify; /* Owner window to send notifications to */ + DWORD dwStyle; + HTREEITEM root; + UINT uInternalStatus; + INT Timer; + UINT uNumItems; /* number of valid TREEVIEW_ITEMs */ + INT cdmode; /* last custom draw setting */ + UINT uScrollTime; /* max. time for scrolling in milliseconds */ + BOOL bRedraw; /* if FALSE we validate but don't redraw in TREEVIEW_Paint() */ + + UINT uItemHeight; /* item height */ + BOOL bHeightSet; + + LONG clientWidth; /* width of control window */ + LONG clientHeight; /* height of control window */ + + LONG treeWidth; /* width of visible tree items */ + LONG treeHeight; /* height of visible tree items */ + + UINT uIndent; /* indentation in pixels */ + HTREEITEM selectedItem; /* handle to selected item or 0 if none */ + HTREEITEM hotItem; /* handle currently under cursor, 0 if none */ + HTREEITEM focusedItem; /* item that was under the cursor when WM_LBUTTONDOWN was received */ + HTREEITEM editItem; /* item being edited with builtin edit box */ + + HTREEITEM firstVisible; /* handle to first visible item */ + LONG maxVisibleOrder; + HTREEITEM dropItem; /* handle to item selected by drag cursor */ + HTREEITEM insertMarkItem; /* item after which insertion mark is placed */ + BOOL insertBeforeorAfter; /* flag used by TVM_SETINSERTMARK */ + HIMAGELIST dragList; /* Bitmap of dragged item */ + LONG scrollX; + COLORREF clrBk; + COLORREF clrText; + COLORREF clrLine; + COLORREF clrInsertMark; + HFONT hFont; + HFONT hDefaultFont; + HFONT hBoldFont; + HFONT hUnderlineFont; + HCURSOR hcurHand; + HWND hwndToolTip; + + HWND hwndEdit; + WNDPROC wpEditOrig; /* orig window proc for subclassing edit */ + BOOL bIgnoreEditKillFocus; + BOOL bLabelChanged; + + BOOL bNtfUnicode; /* TRUE if should send NOTIFY with W */ + HIMAGELIST himlNormal; + int normalImageHeight; + int normalImageWidth; + HIMAGELIST himlState; + int stateImageHeight; + int stateImageWidth; + HDPA items; + + DWORD lastKeyPressTimestamp; + WPARAM charCode; + INT nSearchParamLength; + WCHAR szSearchParam[ MAX_PATH ]; +} TREEVIEW_INFO; typedef struct _TREEITEM /* HTREEITEM is a _TREEINFO *. */ { @@ -101,76 +160,8 @@ LONG textOffset; LONG textWidth; /* horizontal text extent for pszText */ LONG visibleOrder; /* visible ordering, 0 is first visible item */ + const TREEVIEW_INFO *infoPtr; /* tree data this item belongs to */ } TREEVIEW_ITEM; - - -typedef struct tagTREEVIEW_INFO -{ - HWND hwnd; - HWND hwndNotify; /* Owner window to send notifications to */ - DWORD dwStyle; - HTREEITEM root; - UINT uInternalStatus; - INT Timer; - UINT uNumItems; /* number of valid TREEVIEW_ITEMs */ - INT cdmode; /* last custom draw setting */ - UINT uScrollTime; /* max. time for scrolling in milliseconds */ - BOOL bRedraw; /* if FALSE we validate but don't redraw in TREEVIEW_Paint() */ - - UINT uItemHeight; /* item height */ - BOOL bHeightSet; - - LONG clientWidth; /* width of control window */ - LONG clientHeight; /* height of control window */ - - LONG treeWidth; /* width of visible tree items */ - LONG treeHeight; /* height of visible tree items */ - - UINT uIndent; /* indentation in pixels */ - HTREEITEM selectedItem; /* handle to selected item or 0 if none */ - HTREEITEM hotItem; /* handle currently under cursor, 0 if none */ - HTREEITEM focusedItem; /* item that was under the cursor when WM_LBUTTONDOWN was received */ - HTREEITEM editItem; /* item being edited with builtin edit box */ - - HTREEITEM firstVisible; /* handle to first visible item */ - LONG maxVisibleOrder; - HTREEITEM dropItem; /* handle to item selected by drag cursor */ - HTREEITEM insertMarkItem; /* item after which insertion mark is placed */ - BOOL insertBeforeorAfter; /* flag used by TVM_SETINSERTMARK */ - HIMAGELIST dragList; /* Bitmap of dragged item */ - LONG scrollX; - COLORREF clrBk; - COLORREF clrText; - COLORREF clrLine; - COLORREF clrInsertMark; - HFONT hFont; - HFONT hDefaultFont; - HFONT hBoldFont; - HFONT hUnderlineFont; - HCURSOR hcurHand; - HWND hwndToolTip; - - HWND hwndEdit; - WNDPROC wpEditOrig; /* orig window proc for subclassing edit */ - BOOL bIgnoreEditKillFocus; - BOOL bLabelChanged; - - BOOL bNtfUnicode; /* TRUE if should send NOTIFY with W */ - HIMAGELIST himlNormal; - int normalImageHeight; - int normalImageWidth; - HIMAGELIST himlState; - int stateImageHeight; - int stateImageWidth; - enum StateListType statehimlType; - HDPA items; - - DWORD lastKeyPressTimestamp; - WPARAM charCode; - INT nSearchParamLength; - WCHAR szSearchParam[ MAX_PATH ]; -} TREEVIEW_INFO; - /******** Defines that TREEVIEW_ProcessLetterKeys uses ****************/ #define KEY_DELAY 450 @@ -217,7 +208,6 @@ static LRESULT TREEVIEW_DoSelectItem(TREEVIEW_INFO *, INT, HTREEITEM, INT); static VOID TREEVIEW_SetFirstVisible(TREEVIEW_INFO *, TREEVIEW_ITEM *, BOOL); static LRESULT TREEVIEW_EnsureVisible(TREEVIEW_INFO *, HTREEITEM, BOOL); -static LRESULT TREEVIEW_RButtonUp(const TREEVIEW_INFO *, const POINT *); static LRESULT TREEVIEW_EndEditLabelNow(TREEVIEW_INFO *infoPtr, BOOL bCancel); static VOID TREEVIEW_UpdateScrollBars(TREEVIEW_INFO *infoPtr); static LRESULT TREEVIEW_HScroll(TREEVIEW_INFO *, WPARAM); @@ -858,7 +848,9 @@ format = SendMessageW(hwndFrom, WM_NOTIFYFORMAT, (WPARAM)infoPtr->hwnd, NF_QUERY); TRACE("format=%d\n", format); - if (format != NFR_ANSI && format != NFR_UNICODE) return 0; + /* Invalid format returned by NF_QUERY defaults to ANSI*/ + if (format != NFR_ANSI && format != NFR_UNICODE) + format = NFR_ANSI; infoPtr->bNtfUnicode = (format == NFR_UNICODE); @@ -1020,6 +1012,7 @@ newItem->iImage = 0; newItem->iSelectedImage = 0; newItem->iExpandedImage = (WORD)I_IMAGENONE; + newItem->infoPtr = infoPtr; if (DPA_InsertPtr(infoPtr->items, INT_MAX, newItem) == -1) { @@ -1621,7 +1614,7 @@ static LRESULT TREEVIEW_SetRedraw(TREEVIEW_INFO* infoPtr, WPARAM wParam) { - infoPtr->bRedraw = wParam ? TRUE : FALSE; + infoPtr->bRedraw = wParam != 0; if (infoPtr->bRedraw) { @@ -1793,11 +1786,8 @@ infoPtr->himlState = himlNew; if (himlNew) - { ImageList_GetIconSize(himlNew, &infoPtr->stateImageWidth, &infoPtr->stateImageHeight); - infoPtr->statehimlType = OriginUser; - } else { infoPtr->stateImageWidth = 0; @@ -2093,7 +2083,13 @@ TREEVIEW_ITEM *item = tvItem->hItem; if (!TREEVIEW_ValidItem(infoPtr, item)) - return FALSE; + { + if (!item) return FALSE; + + TRACE("got item from different tree %p, called from %p\n", item->infoPtr, infoPtr); + infoPtr = item->infoPtr; + if (!TREEVIEW_ValidItem(infoPtr, item)) return FALSE; + } TREEVIEW_UpdateDispInfo(infoPtr, item, tvItem->mask); @@ -3243,21 +3239,23 @@ RECT scrollRect; LONG scrollDist = 0; TREEVIEW_ITEM *nextItem = NULL, *tmpItem; + BOOL wasExpanded; TRACE("TVE_COLLAPSE %p %s\n", item, TREEVIEW_ItemName(item)); - if (!(item->state & TVIS_EXPANDED)) + if (!TREEVIEW_HasChildren(infoPtr, item)) return FALSE; - if (bUser || !(item->state & TVIS_EXPANDEDONCE)) + if (bUser) TREEVIEW_SendExpanding(infoPtr, item, action); if (item->firstChild == NULL) return FALSE; + wasExpanded = (item->state & TVIS_EXPANDED) != 0; item->state &= ~TVIS_EXPANDED; - if (bUser || !(item->state & TVIS_EXPANDEDONCE)) + if (wasExpanded && bUser) TREEVIEW_SendExpanded(infoPtr, item, action); bSetSelection = (infoPtr->selectedItem != NULL @@ -3338,7 +3336,7 @@ bSetFirstVisible ? item : infoPtr->firstVisible, TRUE); - return TRUE; + return wasExpanded; } static BOOL @@ -3353,8 +3351,8 @@ TRACE("(%p, %p, partial=%d, %d\n", infoPtr, item, partial, user); - if (item->state & TVIS_EXPANDED) - return TRUE; + if (!TREEVIEW_HasChildren(infoPtr, item)) + return FALSE; tmpItem = item; nextItem = NULL; while (tmpItem) @@ -3820,6 +3818,9 @@ TEXTMETRICW textMetric; TRACE("%p %p\n", hwnd, hItem); + if (!(infoPtr->dwStyle & TVS_EDITLABELS)) + return NULL; + if (!TREEVIEW_ValidItem(infoPtr, hItem)) return NULL; @@ -4050,8 +4051,6 @@ else if (msg.message >= WM_LBUTTONDOWN && msg.message <= WM_RBUTTONDBLCLK) { - if (msg.message == WM_RBUTTONUP) - TREEVIEW_RButtonUp(infoPtr, &pt); break; } @@ -4267,33 +4266,16 @@ else { SetFocus(infoPtr->hwnd); - TREEVIEW_SendSimpleNotify(infoPtr, NM_RCLICK); + if(!TREEVIEW_SendSimpleNotify(infoPtr, NM_RCLICK)) + { + /* Send a WM_CONTEXTMENU message in response to the RBUTTONUP */ + SendMessageW(infoPtr->hwndNotify, WM_CONTEXTMENU, + (WPARAM)infoPtr->hwnd, (LPARAM)GetMessagePos()); + } } return 0; } - -static LRESULT -TREEVIEW_RButtonUp(const TREEVIEW_INFO *infoPtr, const POINT *pPt) -{ - TVHITTESTINFO ht; - - ht.pt = *pPt; - - TREEVIEW_HitTest(infoPtr, &ht); - - if (ht.hItem) - { - /* Change to screen coordinate for WM_CONTEXTMENU */ - ClientToScreen(infoPtr->hwnd, &ht.pt); - - /* Send a WM_CONTEXTMENU message in response to the RBUTTONUP */ - SendMessageW(infoPtr->hwnd, WM_CONTEXTMENU, - (WPARAM)infoPtr->hwnd, MAKELPARAM(ht.pt.x, ht.pt.y)); - } - return 0; -} - static LRESULT TREEVIEW_CreateDragImage(TREEVIEW_INFO *infoPtr, LPARAM lParam) @@ -4975,7 +4957,6 @@ int nIndex; infoPtr->himlState = ImageList_Create(16, 16, ILC_COLOR | ILC_MASK, 3, 0); - infoPtr->statehimlType = OriginInternal; hdcScreen = GetDC(0); @@ -5095,10 +5076,7 @@ infoPtr->hwndNotify = lpcs->hwndParent; infoPtr->hwndToolTip = 0; - infoPtr->bNtfUnicode = IsWindowUnicode (hwnd); - - /* Determine what type of notify should be issued */ - /* sets infoPtr->bNtfUnicode */ + /* Determine what type of notify should be issued (sets infoPtr->bNtfUnicode) */ TREEVIEW_NotifyFormat(infoPtr, infoPtr->hwndNotify, NF_REQUERY); if (!(infoPtr->dwStyle & TVS_NOTOOLTIPS)) @@ -5139,8 +5117,6 @@ CloseThemeData (GetWindowTheme (infoPtr->hwnd)); - if (infoPtr->statehimlType == OriginInternal) - ImageList_Destroy(infoPtr->himlState); /* Deassociate treeview from the window before doing anything drastic. */ SetWindowLongPtrW(infoPtr->hwnd, 0, 0); @@ -5269,13 +5245,11 @@ break; case VK_ADD: - if (!(prevItem->state & TVIS_EXPANDED)) - TREEVIEW_Expand(infoPtr, prevItem, FALSE, TRUE); + TREEVIEW_Expand(infoPtr, prevItem, FALSE, TRUE); break; case VK_SUBTRACT: - if (prevItem->state & TVIS_EXPANDED) - TREEVIEW_Collapse(infoPtr, prevItem, FALSE, TRUE); + TREEVIEW_Collapse(infoPtr, prevItem, FALSE, TRUE); break; case VK_PRIOR: Modified: trunk/reactos/include/psdk/commctrl.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/commctrl.h?re… ============================================================================== --- trunk/reactos/include/psdk/commctrl.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/commctrl.h [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -3037,6 +3037,8 @@ #define TVIS_STATEIMAGEMASK 0xF000 #define TVIS_USERMASK 0xF000 +#define TVIS_FOCUSED 0x0001 + #define I_CHILDRENCALLBACK (-1) #define LPTV_ITEMW LPTVITEMW Modified: trunk/reactos/media/doc/README.WINE URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/doc/README.WINE?rev=… ============================================================================== --- trunk/reactos/media/doc/README.WINE [iso-8859-1] (original) +++ trunk/reactos/media/doc/README.WINE [iso-8859-1] Fri Dec 14 23:23:44 2012 @@ -48,7 +48,7 @@ reactos/dll/win32/cabinet # Synced to Wine-1.5.19 reactos/dll/win32/clusapi # Synced to Wine-1.5.19 reactos/dll/win32/comcat # Synced to Wine-1.5.4 -reactos/dll/win32/comctl32 # Synced to Wine 1.3.37 +reactos/dll/win32/comctl32 # Synced to Wine 1.5.19 reactos/dll/win32/comdlg32 # Synced to Wine 1.3.37 reactos/dll/win32/compstui # Synced to Wine-1.5.19 reactos/dll/win32/credui # Synced to Wine-1.5.4

1 0

[akhaldi] 57913: [ROSBE/CMAKE] * Set the correct prefix for the upcoming RosBE 2.1 toolchain.
by akhaldi＠svn.reactos.org 14 Dec '12

14 Dec '12

Author: akhaldi Date: Fri Dec 14 21:56:11 2012 New Revision: 57913 URL: http://svn.reactos.org/svn/reactos?rev=57913&view=rev Log: [ROSBE/CMAKE] * Set the correct prefix for the upcoming RosBE 2.1 toolchain. Modified: trunk/reactos/toolchain-gcc.cmake Modified: trunk/reactos/toolchain-gcc.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/toolchain-gcc.cmake?rev=57… ============================================================================== --- trunk/reactos/toolchain-gcc.cmake [iso-8859-1] (original) +++ trunk/reactos/toolchain-gcc.cmake [iso-8859-1] Fri Dec 14 21:56:11 2012 @@ -10,8 +10,12 @@ if(CMAKE_HOST_SYSTEM_NAME STREQUAL "Windows") set(MINGW_TOOLCHAIN_PREFIX "" CACHE STRING "MinGW Toolchain Prefix") else() - set(MINGW_TOOLCHAIN_PREFIX "mingw32-" CACHE STRING "MinGW Toolchain Prefix") - endif(CMAKE_HOST_SYSTEM_NAME STREQUAL "Windows") + if(NOT $ENV{_ROSBE_VERSION} VERSION_LESS 2.1) + set(MINGW_TOOLCHAIN_PREFIX "i686-w64-mingw32-" CACHE STRING "MinGW-W64 Toolchain Prefix") + else() + set(MINGW_TOOLCHAIN_PREFIX "mingw32-" CACHE STRING "MinGW Toolchain Prefix") + endif() + endif() elseif(ARCH STREQUAL "amd64") set(MINGW_TOOLCHAIN_PREFIX "x86_64-w64-mingw32-" CACHE STRING "MinGW Toolchain Prefix")

1 0

← Newer
1
...
13
14
15
16
17
18
19
...
29
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs December 2012