reform the code.

less warings and errors in vc

svn path=/trunk/; revision=27758
This commit is contained in:
Magnus Olsen 2007-07-20 16:59:44 +00:00
parent 7fd0ba2791
commit a67d29ae20
3 changed files with 46 additions and 44 deletions

View file

@ -293,27 +293,28 @@ NtGdiCreateEnhMetaFile(HDC hDCRef,
LPCWSTR Description) LPCWSTR Description)
{ {
PDC Dc; PDC Dc;
HDC ret = NULL; HDC ret = NULL;
DWORD length = 0; DWORD length = 0;
HDC tempHDC; HDC tempHDC;
DWORD MemSize; DWORD MemSize;
DWORD dwDesiredAccess;
tempHDC = hDCRef; tempHDC = hDCRef;
if (hDCRef == NULL) if (hDCRef == NULL)
{ {
/* FIXME ?? /* FIXME ??
* Shall we create hdc NtGdiHdcCompatible hdc ?? * Shall we create hdc NtGdiHdcCompatible hdc ??
*/ */
UNICODE_STRING DriverName; UNICODE_STRING DriverName;
RtlInitUnicodeString(&DriverName, L"DISPLAY"); RtlInitUnicodeString(&DriverName, L"DISPLAY");
//IntGdiCreateDC(&DriverName, NULL, NULL, NULL, FALSE); //IntGdiCreateDC(&DriverName, NULL, NULL, NULL, FALSE);
tempHDC = NtGdiOpenDCW( &DriverName, tempHDC = NtGdiOpenDCW( &DriverName,
NULL, NULL,
NULL, NULL,
0, // DCW 0 and ICW 1. 0, // DCW 0 and ICW 1.
NULL, NULL,
(PVOID) NULL, (PVOID) NULL,
(PVOID) NULL ); (PVOID) NULL );
} }
GDIOBJ_SetOwnership(GdiHandleTable, tempHDC, PsGetCurrentProcess()); GDIOBJ_SetOwnership(GdiHandleTable, tempHDC, PsGetCurrentProcess());
@ -407,45 +408,44 @@ NtGdiCreateEnhMetaFile(HDC hDCRef,
DPRINT1("Trying Create EnhMetaFile\n"); DPRINT1("Trying Create EnhMetaFile\n");
/* disk based metafile */ /* disk based metafile */
DWORD dwDesiredAccess = GENERIC_WRITE | GENERIC_READ | SYNCHRONIZE | FILE_READ_ATTRIBUTES; dwDesiredAccess = GENERIC_WRITE | GENERIC_READ | SYNCHRONIZE | FILE_READ_ATTRIBUTES;
OBJECT_ATTRIBUTES ObjectAttributes; OBJECT_ATTRIBUTES ObjectAttributes;
IO_STATUS_BLOCK IoStatusBlock; IO_STATUS_BLOCK IoStatusBlock;
IO_STATUS_BLOCK Iosb; IO_STATUS_BLOCK Iosb;
UNICODE_STRING NtPathU; UNICODE_STRING NtPathU;
NTSTATUS Status; NTSTATUS Status;
ULONG FileAttributes = (FILE_ATTRIBUTE_VALID_FLAGS & ~FILE_ATTRIBUTE_DIRECTORY); ULONG FileAttributes = (FILE_ATTRIBUTE_VALID_FLAGS & ~FILE_ATTRIBUTE_DIRECTORY);
if (!RtlDosPathNameToNtPathName_U (File, &NtPathU, NULL, NULL)) if (!RtlDosPathNameToNtPathName_U (File, &NtPathU, NULL, NULL))
{ {
DC_UnlockDc(Dc); DC_UnlockDc(Dc);
if (hDCRef == NULL) if (hDCRef == NULL)
{ {
NtGdiDeleteObjectApp(tempHDC); NtGdiDeleteObjectApp(tempHDC);
} }
DPRINT1("Can not Create EnhMetaFile\n"); DPRINT1("Can not Create EnhMetaFile\n");
SetLastWin32Error(ERROR_PATH_NOT_FOUND); SetLastWin32Error(ERROR_PATH_NOT_FOUND);
return NULL; return NULL;
} }
InitializeObjectAttributes(&ObjectAttributes, &NtPathU, 0, NULL, NULL); InitializeObjectAttributes(&ObjectAttributes, &NtPathU, 0, NULL, NULL);
Status = NtCreateFile (&Dc->hFile, dwDesiredAccess, &ObjectAttributes, &IoStatusBlock, Status = NtCreateFile (&Dc->hFile, dwDesiredAccess, &ObjectAttributes, &IoStatusBlock,
NULL, FileAttributes, 0, FILE_OVERWRITE_IF, FILE_NON_DIRECTORY_FILE, NULL, FileAttributes, 0, FILE_OVERWRITE_IF, FILE_NON_DIRECTORY_FILE,
NULL, 0); NULL, 0);
RtlFreeHeap(RtlGetProcessHeap(), 0, NtPathU.Buffer); RtlFreeHeap(RtlGetProcessHeap(), 0, NtPathU.Buffer);
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
Dc->hFile = NULL; Dc->hFile = NULL;
DC_UnlockDc(Dc); DC_UnlockDc(Dc);
if (hDCRef == NULL) if (hDCRef == NULL)
{ {
NtGdiDeleteObjectApp(tempHDC); NtGdiDeleteObjectApp(tempHDC);
} }
DPRINT1("Create EnhMetaFile fail\n"); DPRINT1("Create EnhMetaFile fail\n");
SetLastWin32Error(ERROR_INVALID_HANDLE); SetLastWin32Error(ERROR_INVALID_HANDLE);
return NULL; return NULL;
} }
@ -462,20 +462,20 @@ NtGdiCreateEnhMetaFile(HDC hDCRef,
} }
if (NT_SUCCESS(Status)) if (NT_SUCCESS(Status))
{ {
ret = tempHDC; ret = tempHDC;
DC_UnlockDc(Dc); DC_UnlockDc(Dc);
} }
else else
{ {
Dc->hFile = NULL; Dc->hFile = NULL;
DPRINT1("Write to EnhMetaFile fail\n"); DPRINT1("Write to EnhMetaFile fail\n");
SetLastWin32Error(ERROR_CAN_NOT_COMPLETE); SetLastWin32Error(ERROR_CAN_NOT_COMPLETE);
ret = NULL; ret = NULL;
DC_UnlockDc(Dc); DC_UnlockDc(Dc);
if (hDCRef == NULL) if (hDCRef == NULL)
{ {
NtGdiDeleteObjectApp(tempHDC); NtGdiDeleteObjectApp(tempHDC);
} }
} }
} }
@ -483,8 +483,8 @@ NtGdiCreateEnhMetaFile(HDC hDCRef,
{ {
DC_UnlockDc(Dc); DC_UnlockDc(Dc);
} }
return ret; return ret;
} }

View file

@ -212,9 +212,7 @@ NtGdiGetPath(
{ {
INT ret = -1; INT ret = -1;
GdiPath *pPath; GdiPath *pPath;
DPRINT("Enter NtGdiGetPath\n");
DC *dc = DC_LockDc(hDC); DC *dc = DC_LockDc(hDC);
if(!dc) if(!dc)
{ {

View file

@ -2502,6 +2502,8 @@ NtGdiGetGlyphOutline(
INT adv, lsb, bbx; /* These three hold to widths of the unrotated chars */ INT adv, lsb, bbx; /* These three hold to widths of the unrotated chars */
OUTLINETEXTMETRICW *potm; OUTLINETEXTMETRICW *potm;
PVOID pvBuf = NULL; PVOID pvBuf = NULL;
int n = 0;
FT_CharMap found = 0, charmap;
DPRINT("%p, %d, %08x, %p, %08lx, %p, %p\n", hdc, wch, iFormat, pgm, DPRINT("%p, %d, %08x, %p, %08lx, %p, %p\n", hdc, wch, iFormat, pgm,
cjBuf, UnsafeBuf, pmat2); cjBuf, UnsafeBuf, pmat2);
@ -2544,8 +2546,8 @@ NtGdiGetGlyphOutline(
{ {
DPRINT("WARNING: No charmap selected!\n"); DPRINT("WARNING: No charmap selected!\n");
DPRINT("This font face has %d charmaps\n", ft_face->num_charmaps); DPRINT("This font face has %d charmaps\n", ft_face->num_charmaps);
int n;
FT_CharMap found = 0, charmap;
for (n = 0; n < ft_face->num_charmaps; n++) for (n = 0; n < ft_face->num_charmaps; n++)
{ {
@ -4159,13 +4161,15 @@ NtGdiGetSetTextCharExtra( HDC hDC, INT CharExtra, BOOL Set)
{ {
/* Ulta-Ugly Hax! */ /* Ulta-Ugly Hax! */
INT Ret = 0x80000000; INT Ret = 0x80000000;
DPRINT("TextCharacterExtra %d", CharExtra);
PDC dc = DC_LockDc ( hDC ); PDC dc = DC_LockDc ( hDC );
if (!dc) if (!dc)
{ {
DPRINT("TextCharacterExtra %d", CharExtra);
SetLastWin32Error(ERROR_INVALID_HANDLE); SetLastWin32Error(ERROR_INVALID_HANDLE);
return Ret; return Ret;
} }
DPRINT("TextCharacterExtra %d", CharExtra);
Ret = dc->w.charExtra; Ret = dc->w.charExtra;
if( Set ) dc->w.charExtra = CharExtra; if( Set ) dc->w.charExtra = CharExtra;
DC_UnlockDc(dc); DC_UnlockDc(dc);