- 09 Feb, 2009 40 commits
-
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
The latter was unused while the former did not seem very useful.
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Francois Gouget authored
-
Dylan Smith authored
These functions were just being used for addition, so it was simpler to remove the functions and modify the places it was used. The ME_StrRelPos2 and ME_PosToVPos were just simple wrappers around ME_StrRelPos, and ME_PosToVPos wasn't being used.
-
Dylan Smith authored
These two functions were being used for simple operations, to get the first or last character when pre-computing flags for splitting runs. The call to ME_GetCharBack wasn't even giving the correct result, it would always return -1 since it is being called with nPos of 0. This patch simplifies the code by removing the functions and getting the characters directly from the string.
-
Dylan Smith authored
These functions were probably previously needed because of some wierd special handling of backspace characters, but currently there is no reason why the nLen field can't be accessed directly. Having to functions that just access the string length field just causes slightly more effort for someone to look at the code, because they need to enter the function to find out what it actually is doing.
-
Dylan Smith authored
The function was just returning the second parameter. It had some commented out code that indicated that previously backslashes weren't included in the length. Native wordpad doesn't handle backspaces in a special way, so this must have been an internal representation that complicated finding the position of characters.
-
Dylan Smith authored
ME_GetCursorCoordinates had two conditions that were always taken. The first condition was if(pCursor->pRun->type == diRun) was following an assertion making the exact same check. The next one, if(row), should always be taken, otherwise the richedit controls are in a corrupt state, therefore an assertion is more appropriate.
-
Dylan Smith authored
I found that ME_MakeStringB was previously unused, and that the other ME_MakeString functions repeated code that was already in ME_MakeStringB. Making ME_MakeStringB static and using it to avoid duplicate code seemed like a better idea than removing the function.
-
Dylan Smith authored
-
Dylan Smith authored
-
Dylan Smith authored
Previously it found the start or end by traversing the linked lists of run, rows, paragraphs, and cells from the current position of the cursors. Clearly it is better to get the start or end directly to make it a constant time operation.
-
Dylan Smith authored
Wrapping is needed to be done even when repainting isn't done since later messages expect line breaks to reflect the current text. Some message can specify not to paint the sceen, but this should prevent wrapping from being done.
-
Alexandre Julliard authored
-
Alexandre Julliard authored
-
Alexandre Julliard authored
-
Alexandre Julliard authored
-
Alexandre Julliard authored
-
Andrew Talbot authored
-
Andrew Talbot authored
-
Juan Lang authored
-
Juan Lang authored
-
Juan Lang authored
-
Juan Lang authored
cryptui: Ensure a certificate's private key is exportable before allowing it to be selected for export.
-
Juan Lang authored
-
Juan Lang authored
-
Juan Lang authored
cryptui: Skip password page when moving backward through the export wizard if it wasn't shown in the first place.
-
Juan Lang authored
-
Juan Lang authored
-