- Wine-Devel - list.winehq.org

[PATCH 2/2] wineps: Build with nameless structs and unions.
by Huw Davies Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/wineps.drv/driver.c | 84 +++++++++++++++++++--------------------- dlls/wineps.drv/init.c | 55 +++++++++++--------------- dlls/wineps.drv/ps.c | 12 +++--- 3 files changed, 68 insertions(+), 83 deletions(-) diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c index 64ed5bbb402..6d083b2a8c7 100644 --- a/dlls/wineps.drv/driver.c +++ b/dlls/wineps.drv/driver.c @@ -22,9 +22,6 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - #include <stdarg.h> #include "windef.h" @@ -53,7 +50,7 @@ INPUTSLOT *find_slot( PPD *ppd, const PSDRV_DEVMODE *dm ) INPUTSLOT *slot; LIST_FOR_EACH_ENTRY( slot, &ppd->InputSlots, INPUTSLOT, entry ) - if (slot->WinBin == dm->dmPublic.u1.s1.dmDefaultSource) + if (slot->WinBin == dm->dmPublic.dmDefaultSource) return slot; return NULL; @@ -64,7 +61,7 @@ PAGESIZE *find_pagesize( PPD *ppd, const PSDRV_DEVMODE *dm ) PAGESIZE *page; LIST_FOR_EACH_ENTRY( page, &ppd->PageSizes, PAGESIZE, entry ) - if (page->WinPage == dm->dmPublic.u1.s1.dmPaperSize) + if (page->WinPage == dm->dmPublic.dmPaperSize) return page; return NULL; @@ -96,12 +93,12 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI /* some sanity checks here on dm2 */ if(dm2->dmPublic.dmFields & DM_ORIENTATION) { - dm1->dmPublic.u1.s1.dmOrientation = dm2->dmPublic.u1.s1.dmOrientation; + dm1->dmPublic.dmOrientation = dm2->dmPublic.dmOrientation; TRACE("Changing orientation to %d (%s)\n", - dm1->dmPublic.u1.s1.dmOrientation, - dm1->dmPublic.u1.s1.dmOrientation == DMORIENT_PORTRAIT ? + dm1->dmPublic.dmOrientation, + dm1->dmPublic.dmOrientation == DMORIENT_PORTRAIT ? "Portrait" : - (dm1->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE ? + (dm1->dmPublic.dmOrientation == DMORIENT_LANDSCAPE ? "Landscape" : "unknown")); } @@ -112,13 +109,13 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI if (page) { - dm1->dmPublic.u1.s1.dmPaperSize = dm2->dmPublic.u1.s1.dmPaperSize; - dm1->dmPublic.u1.s1.dmPaperWidth = paper_size_from_points( page->PaperDimension->x ); - dm1->dmPublic.u1.s1.dmPaperLength = paper_size_from_points( page->PaperDimension->y ); + dm1->dmPublic.dmPaperSize = dm2->dmPublic.dmPaperSize; + dm1->dmPublic.dmPaperWidth = paper_size_from_points( page->PaperDimension->x ); + dm1->dmPublic.dmPaperLength = paper_size_from_points( page->PaperDimension->y ); dm1->dmPublic.dmFields |= DM_PAPERSIZE | DM_PAPERWIDTH | DM_PAPERLENGTH; TRACE("Changing page to %s %d x %d\n", page->FullName, - dm1->dmPublic.u1.s1.dmPaperWidth, - dm1->dmPublic.u1.s1.dmPaperLength ); + dm1->dmPublic.dmPaperWidth, + dm1->dmPublic.dmPaperLength ); if (dm1->dmPublic.dmSize >= FIELD_OFFSET(DEVMODEW, dmFormName) + CCHFORMNAME * sizeof(WCHAR)) { @@ -127,16 +124,16 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI } } else - TRACE("Trying to change to unsupported pagesize %d\n", dm2->dmPublic.u1.s1.dmPaperSize); + TRACE("Trying to change to unsupported pagesize %d\n", dm2->dmPublic.dmPaperSize); } else if((dm2->dmPublic.dmFields & DM_PAPERLENGTH) && (dm2->dmPublic.dmFields & DM_PAPERWIDTH)) { - dm1->dmPublic.u1.s1.dmPaperLength = dm2->dmPublic.u1.s1.dmPaperLength; - dm1->dmPublic.u1.s1.dmPaperWidth = dm2->dmPublic.u1.s1.dmPaperWidth; + dm1->dmPublic.dmPaperLength = dm2->dmPublic.dmPaperLength; + dm1->dmPublic.dmPaperWidth = dm2->dmPublic.dmPaperWidth; TRACE("Changing PaperLength|Width to %dx%d\n", - dm2->dmPublic.u1.s1.dmPaperLength, - dm2->dmPublic.u1.s1.dmPaperWidth); + dm2->dmPublic.dmPaperLength, + dm2->dmPublic.dmPaperWidth); dm1->dmPublic.dmFields &= ~DM_PAPERSIZE; dm1->dmPublic.dmFields |= (DM_PAPERLENGTH | DM_PAPERWIDTH); } else if(dm2->dmPublic.dmFields & (DM_PAPERLENGTH | DM_PAPERWIDTH)) { @@ -149,13 +146,13 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI } if(dm2->dmPublic.dmFields & DM_SCALE) { - dm1->dmPublic.u1.s1.dmScale = dm2->dmPublic.u1.s1.dmScale; - TRACE("Changing Scale to %d\n", dm2->dmPublic.u1.s1.dmScale); + dm1->dmPublic.dmScale = dm2->dmPublic.dmScale; + TRACE("Changing Scale to %d\n", dm2->dmPublic.dmScale); } if(dm2->dmPublic.dmFields & DM_COPIES) { - dm1->dmPublic.u1.s1.dmCopies = dm2->dmPublic.u1.s1.dmCopies; - TRACE("Changing Copies to %d\n", dm2->dmPublic.u1.s1.dmCopies); + dm1->dmPublic.dmCopies = dm2->dmPublic.dmCopies; + TRACE("Changing Copies to %d\n", dm2->dmPublic.dmCopies); } if (dm2->dmPublic.dmFields & DM_DEFAULTSOURCE) @@ -164,17 +161,17 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI if (slot) { - dm1->dmPublic.u1.s1.dmDefaultSource = dm2->dmPublic.u1.s1.dmDefaultSource; + dm1->dmPublic.dmDefaultSource = dm2->dmPublic.dmDefaultSource; TRACE("Changing bin to '%s'\n", slot->FullName); } else - TRACE("Trying to change to unsupported bin %d\n", dm2->dmPublic.u1.s1.dmDefaultSource); + TRACE("Trying to change to unsupported bin %d\n", dm2->dmPublic.dmDefaultSource); } if (dm2->dmPublic.dmFields & DM_DEFAULTSOURCE ) - dm1->dmPublic.u1.s1.dmDefaultSource = dm2->dmPublic.u1.s1.dmDefaultSource; + dm1->dmPublic.dmDefaultSource = dm2->dmPublic.dmDefaultSource; if (dm2->dmPublic.dmFields & DM_PRINTQUALITY ) - dm1->dmPublic.u1.s1.dmPrintQuality = dm2->dmPublic.u1.s1.dmPrintQuality; + dm1->dmPublic.dmPrintQuality = dm2->dmPublic.dmPrintQuality; if (dm2->dmPublic.dmFields & DM_COLOR ) dm1->dmPublic.dmColor = dm2->dmPublic.dmColor; if (dm2->dmPublic.dmFields & DM_DUPLEX && pi->ppd->DefaultDuplex && pi->ppd->DefaultDuplex->WinDuplex != 0) @@ -194,11 +191,11 @@ void PSDRV_MergeDevmodes( PSDRV_DEVMODE *dm1, const PSDRV_DEVMODE *dm2, PRINTERI if (dm2->dmPublic.dmFields & DM_PELSHEIGHT ) dm1->dmPublic.dmPelsHeight = dm2->dmPublic.dmPelsHeight; if (dm2->dmPublic.dmFields & DM_DISPLAYFLAGS ) - dm1->dmPublic.u2.dmDisplayFlags = dm2->dmPublic.u2.dmDisplayFlags; + dm1->dmPublic.dmDisplayFlags = dm2->dmPublic.dmDisplayFlags; if (dm2->dmPublic.dmFields & DM_DISPLAYFREQUENCY ) dm1->dmPublic.dmDisplayFrequency = dm2->dmPublic.dmDisplayFrequency; if (dm2->dmPublic.dmFields & DM_POSITION ) - dm1->dmPublic.u1.s2.dmPosition = dm2->dmPublic.u1.s2.dmPosition; + dm1->dmPublic.dmPosition = dm2->dmPublic.dmPosition; if (dm2->dmPublic.dmFields & DM_LOGPIXELS ) dm1->dmPublic.dmLogPixels = dm2->dmPublic.dmLogPixels; if (dm2->dmPublic.dmFields & DM_ICMMETHOD ) @@ -247,14 +244,14 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg, LIST_FOR_EACH_ENTRY(ps, &di->pi->ppd->PageSizes, PAGESIZE, entry) { SendDlgItemMessageA(hwnd, IDD_PAPERS, LB_INSERTSTRING, i, (LPARAM)ps->FullName); - if(di->pi->Devmode->dmPublic.u1.s1.dmPaperSize == ps->WinPage) + if(di->pi->Devmode->dmPublic.dmPaperSize == ps->WinPage) Cursel = i; i++; } SendDlgItemMessageA(hwnd, IDD_PAPERS, LB_SETCURSEL, Cursel, 0); CheckRadioButton(hwnd, IDD_ORIENT_PORTRAIT, IDD_ORIENT_LANDSCAPE, - di->pi->Devmode->dmPublic.u1.s1.dmOrientation == + di->pi->Devmode->dmPublic.dmOrientation == DMORIENT_PORTRAIT ? IDD_ORIENT_PORTRAIT : IDD_ORIENT_LANDSCAPE); @@ -298,7 +295,7 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg, resx = resy = di->pi->ppd->DefaultResolution; if (di->pi->Devmode->dmPublic.dmFields & DM_PRINTQUALITY) - resx = resy = di->pi->Devmode->dmPublic.u1.s1.dmPrintQuality; + resx = resy = di->pi->Devmode->dmPublic.dmPrintQuality; if (di->pi->Devmode->dmPublic.dmFields & DM_YRESOLUTION) resy = di->pi->Devmode->dmPublic.dmYResolution; @@ -342,11 +339,11 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg, } TRACE("Setting pagesize to item %d, WinPage %d (%s), PaperSize %.2fx%.2f\n", Cursel, ps->WinPage, ps->FullName, ps->PaperDimension->x, ps->PaperDimension->y); - di->dlgdm->dmPublic.u1.s1.dmPaperSize = ps->WinPage; + di->dlgdm->dmPublic.dmPaperSize = ps->WinPage; di->dlgdm->dmPublic.dmFields |= DM_PAPERSIZE; - di->dlgdm->dmPublic.u1.s1.dmPaperWidth = paper_size_from_points(ps->PaperDimension->x); - di->dlgdm->dmPublic.u1.s1.dmPaperLength = paper_size_from_points(ps->PaperDimension->y); + di->dlgdm->dmPublic.dmPaperWidth = paper_size_from_points(ps->PaperDimension->x); + di->dlgdm->dmPublic.dmPaperLength = paper_size_from_points(ps->PaperDimension->y); di->dlgdm->dmPublic.dmFields |= DM_PAPERLENGTH | DM_PAPERWIDTH; if (di->dlgdm->dmPublic.dmSize >= FIELD_OFFSET(DEVMODEW, dmFormName) + CCHFORMNAME * sizeof(WCHAR)) @@ -361,7 +358,7 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg, case IDD_ORIENT_LANDSCAPE: TRACE("Setting orientation to %s\n", wParam == IDD_ORIENT_PORTRAIT ? "portrait" : "landscape"); - di->dlgdm->dmPublic.u1.s1.dmOrientation = wParam == IDD_ORIENT_PORTRAIT ? + di->dlgdm->dmPublic.dmOrientation = wParam == IDD_ORIENT_PORTRAIT ? DMORIENT_PORTRAIT : DMORIENT_LANDSCAPE; di->dlgdm->dmPublic.dmFields |= DM_ORIENTATION; SendMessageW(GetParent(hwnd), PSM_CHANGED, 0, 0); @@ -395,7 +392,7 @@ static INT_PTR CALLBACK PSDRV_PaperDlgProc(HWND hwnd, UINT msg, resy = HIWORD(data); TRACE("Setting resolution to %dx%d\n", resx, resy); - di->dlgdm->dmPublic.u1.s1.dmPrintQuality = resx; + di->dlgdm->dmPublic.dmPrintQuality = resx; di->dlgdm->dmPublic.dmFields |= DM_PRINTQUALITY; di->dlgdm->dmPublic.dmYResolution = resy; @@ -449,7 +446,7 @@ static int (WINAPI *pPropertySheet) (LPCPROPSHEETHEADERW); * Returns size of DEVMODE structure if wMode is 0. Otherwise, IDOK is returned for success * for both dialog and non-dialog operations. IDCANCEL is returned if the dialog box was cancelled. * A return value less than zero is returned if a non-dialog operation fails. - * + * * BUGS * * Just returns default devmode at the moment. No use of initialization file. @@ -496,8 +493,8 @@ INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *outpu di.dlgdm = &dlgdm; psp.dwSize = sizeof(psp); psp.hInstance = PSDRV_hInstance; - psp.u.pszTemplate = L"PAPER"; - psp.u2.pszIcon = NULL; + psp.pszTemplate = L"PAPER"; + psp.pszIcon = NULL; psp.pfnDlgProc = PSDRV_PaperDlgProc; psp.lParam = (LPARAM)&di; hpsp[0] = pCreatePropertySheetPage(&psp); @@ -507,12 +504,12 @@ INT WINAPI DrvDocumentProperties(HWND hwnd, const WCHAR *device, DEVMODEW *outpu psh.pszCaption = SetupW; psh.nPages = 1; psh.hwndParent = hwnd; - psh.u3.phpage = hpsp; + psh.phpage = hpsp; pPropertySheet(&psh); } - + /* If DM_UPDATE is set, should write settings to environment and initialization file */ if (mode & DM_UPDATE) FIXME("Mode DM_UPDATE. Just do the same as DM_COPY\n"); @@ -814,7 +811,6 @@ DWORD WINAPI DrvDeviceCapabilities(HANDLE printer, WCHAR *device_name, WORD capa * languages supported by the printer, unless pOutput is NULL. The return value is * number of array entries. (Win2k/XP only) */ - case DC_PERSONALITY: /* WINVER >= 0x0500 */ FIXME("DC_PERSONALITY: stub\n"); ret = -1; @@ -840,7 +836,7 @@ DWORD WINAPI DrvDeviceCapabilities(HANDLE printer, WCHAR *device_name, WORD capa /* Returns the printer rate unit used for DC_PRINTRATE, which is one of * PRINTRATEUNIT_{CPS,IPM,LPM,PPM} (Win2k/XP only) - */ + */ case DC_PRINTRATEUNIT: /* WINVER >= 0x0500 */ FIXME("DC_PRINTRATEUNIT: stub\n"); ret = -1; diff --git a/dlls/wineps.drv/init.c b/dlls/wineps.drv/init.c index d88a984e210..934c378eeef 100644 --- a/dlls/wineps.drv/init.c +++ b/dlls/wineps.drv/init.c @@ -22,9 +22,6 @@ #include <stdarg.h> #include <string.h> -#define NONAMELESSUNION -#define NONAMELESSSTRUCT - #include "windef.h" #include "winbase.h" #include "winerror.h" @@ -210,14 +207,14 @@ static void dump_devmode(const DEVMODEW *dm) TRACE("dmDriverExtra: 0x%04x\n", dm->dmDriverExtra); TRACE("dmFields: 0x%04x\n", dm->dmFields); dump_fields(dm->dmFields); - TRACE("dmOrientation: %d\n", dm->u1.s1.dmOrientation); - TRACE("dmPaperSize: %d\n", dm->u1.s1.dmPaperSize); - TRACE("dmPaperLength: %d\n", dm->u1.s1.dmPaperLength); - TRACE("dmPaperWidth: %d\n", dm->u1.s1.dmPaperWidth); - TRACE("dmScale: %d\n", dm->u1.s1.dmScale); - TRACE("dmCopies: %d\n", dm->u1.s1.dmCopies); - TRACE("dmDefaultSource: %d\n", dm->u1.s1.dmDefaultSource); - TRACE("dmPrintQuality: %d\n", dm->u1.s1.dmPrintQuality); + TRACE("dmOrientation: %d\n", dm->dmOrientation); + TRACE("dmPaperSize: %d\n", dm->dmPaperSize); + TRACE("dmPaperLength: %d\n", dm->dmPaperLength); + TRACE("dmPaperWidth: %d\n", dm->dmPaperWidth); + TRACE("dmScale: %d\n", dm->dmScale); + TRACE("dmCopies: %d\n", dm->dmCopies); + TRACE("dmDefaultSource: %d\n", dm->dmDefaultSource); + TRACE("dmPrintQuality: %d\n", dm->dmPrintQuality); TRACE("dmColor: %d\n", dm->dmColor); TRACE("dmDuplex: %d\n", dm->dmDuplex); TRACE("dmYResolution: %d\n", dm->dmYResolution); @@ -241,7 +238,7 @@ static void PSDRV_UpdateDevCaps( PSDRV_PDEVICE *physDev ) if (physDev->Devmode->dmPublic.dmFields & (DM_PRINTQUALITY | DM_YRESOLUTION | DM_LOGPIXELS)) { if (physDev->Devmode->dmPublic.dmFields & DM_PRINTQUALITY) - resx = resy = physDev->Devmode->dmPublic.u1.s1.dmPrintQuality; + resx = resy = physDev->Devmode->dmPublic.dmPrintQuality; if (physDev->Devmode->dmPublic.dmFields & DM_YRESOLUTION) resy = physDev->Devmode->dmPublic.dmYResolution; @@ -275,7 +272,7 @@ static void PSDRV_UpdateDevCaps( PSDRV_PDEVICE *physDev ) if(physDev->Devmode->dmPublic.dmFields & DM_PAPERSIZE) { LIST_FOR_EACH_ENTRY(page, &physDev->pi->ppd->PageSizes, PAGESIZE, entry) { - if(page->WinPage == physDev->Devmode->dmPublic.u1.s1.dmPaperSize) + if(page->WinPage == physDev->Devmode->dmPublic.dmPaperSize) break; } @@ -306,11 +303,9 @@ static void PSDRV_UpdateDevCaps( PSDRV_PDEVICE *physDev ) /* physDev sizes in device units; Devmode sizes in 1/10 mm */ physDev->ImageableArea.left = physDev->ImageableArea.bottom = 0; physDev->ImageableArea.right = physDev->PageSize.cx = - physDev->Devmode->dmPublic.u1.s1.dmPaperWidth * - physDev->logPixelsX / 254; + physDev->Devmode->dmPublic.dmPaperWidth * physDev->logPixelsX / 254; physDev->ImageableArea.top = physDev->PageSize.cy = - physDev->Devmode->dmPublic.u1.s1.dmPaperLength * - physDev->logPixelsY / 254; + physDev->Devmode->dmPublic.dmPaperLength * physDev->logPixelsY / 254; } else { FIXME("Odd dmFields %x\n", physDev->Devmode->dmPublic.dmFields); SetRectEmpty(&physDev->ImageableArea); @@ -325,7 +320,7 @@ static void PSDRV_UpdateDevCaps( PSDRV_PDEVICE *physDev ) width = physDev->ImageableArea.right - physDev->ImageableArea.left; height = physDev->ImageableArea.top - physDev->ImageableArea.bottom; - if(physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_PORTRAIT) { + if(physDev->Devmode->dmPublic.dmOrientation == DMORIENT_PORTRAIT) { physDev->horzRes = width; physDev->vertRes = height; } else { @@ -475,11 +470,9 @@ static INT CDECL PSDRV_GetDeviceCaps( PHYSDEV dev, INT cap ) case TECHNOLOGY: return DT_RASPRINTER; case HORZSIZE: - return MulDiv(physDev->horzSize, 100, - physDev->Devmode->dmPublic.u1.s1.dmScale); + return MulDiv(physDev->horzSize, 100, physDev->Devmode->dmPublic.dmScale); case VERTSIZE: - return MulDiv(physDev->vertSize, 100, - physDev->Devmode->dmPublic.u1.s1.dmScale); + return MulDiv(physDev->vertSize, 100, physDev->Devmode->dmPublic.dmScale); case HORZRES: return physDev->horzRes; case VERTRES: @@ -506,23 +499,21 @@ static INT CDECL PSDRV_GetDeviceCaps( PHYSDEV dev, INT cap ) case ASPECTY: return physDev->logPixelsY; case LOGPIXELSX: - return MulDiv(physDev->logPixelsX, - physDev->Devmode->dmPublic.u1.s1.dmScale, 100); + return MulDiv(physDev->logPixelsX, physDev->Devmode->dmPublic.dmScale, 100); case LOGPIXELSY: - return MulDiv(physDev->logPixelsY, - physDev->Devmode->dmPublic.u1.s1.dmScale, 100); + return MulDiv(physDev->logPixelsY, physDev->Devmode->dmPublic.dmScale, 100); case NUMRESERVED: return 0; case COLORRES: return 0; case PHYSICALWIDTH: - return (physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE) ? + return (physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE) ? physDev->PageSize.cy : physDev->PageSize.cx; case PHYSICALHEIGHT: - return (physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE) ? + return (physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE) ? physDev->PageSize.cx : physDev->PageSize.cy; case PHYSICALOFFSETX: - if(physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE) { + if(physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE) { if(physDev->pi->ppd->LandscapeOrientation == -90) return physDev->PageSize.cy - physDev->ImageableArea.top; else @@ -531,7 +522,7 @@ static INT CDECL PSDRV_GetDeviceCaps( PHYSDEV dev, INT cap ) return physDev->ImageableArea.left; case PHYSICALOFFSETY: - if(physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE) { + if(physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE) { if(physDev->pi->ppd->LandscapeOrientation == -90) return physDev->PageSize.cx - physDev->ImageableArea.right; else @@ -709,7 +700,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) PSDRV_DEVMODE dm; memset(&dm, 0, sizeof(dm)); dm.dmPublic.dmFields = DM_PAPERSIZE; - dm.dmPublic.u1.s1.dmPaperSize = papersize; + dm.dmPublic.dmPaperSize = papersize; PSDRV_MergeDevmodes(pi->Devmode, &dm, pi); } } @@ -718,7 +709,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) PSDRV_DEVMODE dm; memset(&dm, 0, sizeof(dm)); dm.dmPublic.dmFields = DM_PAPERSIZE; - dm.dmPublic.u1.s1.dmPaperSize = pi->ppd->DefaultPageSize->WinPage; + dm.dmPublic.dmPaperSize = pi->ppd->DefaultPageSize->WinPage; PSDRV_MergeDevmodes(pi->Devmode, &dm, pi); } diff --git a/dlls/wineps.drv/ps.c b/dlls/wineps.drv/ps.c index 60737d67bda..f88d44a829d 100644 --- a/dlls/wineps.drv/ps.c +++ b/dlls/wineps.drv/ps.c @@ -25,8 +25,6 @@ #include <stdarg.h> #include <locale.h> -#define NONAMELESSUNION -#define NONAMELESSSTRUCT #include "windef.h" #include "winbase.h" #include "winnls.h" @@ -381,10 +379,10 @@ static void write_cups_job_ticket( PHYSDEV dev, const struct ticket_info *info ) } } - if (physDev->Devmode->dmPublic.u1.s1.dmCopies > 1) + if (physDev->Devmode->dmPublic.dmCopies > 1) { len = snprintf( buf, sizeof(buf), "%%cupsJobTicket: copies=%d\n", - physDev->Devmode->dmPublic.u1.s1.dmCopies ); + physDev->Devmode->dmPublic.dmCopies ); if (len > 0 && len < sizeof(buf)) write_spool( dev, buf, len ); @@ -398,7 +396,7 @@ static void write_cups_job_ticket( PHYSDEV dev, const struct ticket_info *info ) } if (!(physDev->Devmode->dmPublic.dmFields & DM_DEFAULTSOURCE) || - physDev->Devmode->dmPublic.u1.s1.dmDefaultSource == DMBIN_AUTO) + physDev->Devmode->dmPublic.dmDefaultSource == DMBIN_AUTO) write_spool( dev, cups_ap_d_inputslot, sizeof(cups_ap_d_inputslot) - 1 ); } @@ -444,7 +442,7 @@ INT PSDRV_WriteHeader( PHYSDEV dev, LPCWSTR title ) ury = physDev->ImageableArea.top * 72.0 / physDev->logPixelsY; /* FIXME should do something better with BBox */ - dmOrientation = (physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE ? "Landscape" : "Portrait"); + dmOrientation = (physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE ? "Landscape" : "Portrait"); sprintf(buf, psheader, escaped_title, llx, lly, urx, ury, dmOrientation); HeapFree(GetProcessHeap(), 0, escaped_title); @@ -526,7 +524,7 @@ INT PSDRV_WriteNewPage( PHYSDEV dev ) return 0; } - if(physDev->Devmode->dmPublic.u1.s1.dmOrientation == DMORIENT_LANDSCAPE) { + if(physDev->Devmode->dmPublic.dmOrientation == DMORIENT_LANDSCAPE) { if(physDev->pi->ppd->LandscapeOrientation == -90) { xtrans = physDev->ImageableArea.right; ytrans = physDev->ImageableArea.top; -- 2.23.0

1 0

[PATCH 1/2] wineps: Return a POINT for DC_PAPERSIZE.
by Huw Davies Dec. 6, 2021

Dec. 6, 2021

From: Konstantin Romanov <incubusrk(a)gmail.com> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52132 Signed-off-by: Huw Davies <huw(a)codeweavers.com> --- dlls/wineps.drv/driver.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c index 11de7bc3328..64ed5bbb402 100644 --- a/dlls/wineps.drv/driver.c +++ b/dlls/wineps.drv/driver.c @@ -25,15 +25,19 @@ #define NONAMELESSUNION #define NONAMELESSSTRUCT -#include <string.h> - -#include "wine/debug.h" -#include "psdrv.h" -#include "ddk/winddiui.h" +#include <stdarg.h> +#include "windef.h" +#include "winbase.h" +#include "winnls.h" +#include "wingdi.h" #include "winuser.h" -#include "wine/wingdi16.h" #include "prsht.h" +#include "ddk/winddiui.h" + +#include "wine/debug.h" + +#include "psdrv.h" #include "psdlg.h" WINE_DEFAULT_DEBUG_CHANNEL(psdrv); @@ -561,7 +565,7 @@ DWORD WINAPI DrvDeviceCapabilities(HANDLE printer, WCHAR *device_name, WORD capa case DC_PAPERSIZE: { PAGESIZE *ps; - POINT16 *pt = output; + POINT *pt = output; int i = 0; LIST_FOR_EACH_ENTRY(ps, &pi->ppd->PageSizes, PAGESIZE, entry) -- 2.23.0

1 0

[PATCH 10/10] win32u: Directly use sysparams in nulldrv_GetDeviceCaps.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/win32u/driver.c | 50 +++++++----------------------------- dlls/win32u/ntgdi_private.h | 1 - dlls/win32u/sysparams.c | 20 ++++++++++++++- dlls/win32u/win32u_private.h | 5 ++-- dlls/win32u/wrappers.c | 28 -------------------- 5 files changed, 30 insertions(+), 74 deletions(-)

2 1

[PATCH 09/10] win32u: Move system parameters implementation from user32.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- I tried to reduce this patch's size with prvious patches (and previous series), but in the end, I think that it's best to move all params at the time. We need at least default_entries to be moved together and on top of that, further splitting would be a small win, probably not worth additional mess. dlls/user32/driver.c | 8 +- dlls/user32/message.c | 2 +- dlls/user32/sysparams.c | 2549 ++-------------------------------- dlls/user32/user32.spec | 4 +- dlls/win32u/driver.c | 5 +- dlls/win32u/gdiobj.c | 3 + dlls/win32u/syscall.c | 1 + dlls/win32u/sysparams.c | 2338 ++++++++++++++++++++++++++++++- dlls/win32u/win32u.spec | 8 +- dlls/win32u/win32u_private.h | 7 +- dlls/win32u/wrappers.c | 28 + dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 5 + include/ntuser.h | 11 + 14 files changed, 2440 insertions(+), 2530 deletions(-)

2 1

[PATCH 08/10] win32u: Directly access system DPI in ntgdi.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/win32u/gdiobj.c | 10 ---------- dlls/win32u/sysparams.c | 7 +++++++ dlls/win32u/win32u_private.h | 2 +- dlls/win32u/wrappers.c | 9 --------- 4 files changed, 8 insertions(+), 20 deletions(-)

2 1

[PATCH 07/10] win32u: Move DPI mapping from EnumDisplayMonitors.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/sysparams.c | 51 ++++++----------------------------------- dlls/user32/user32.spec | 2 +- dlls/win32u/sysparams.c | 29 ++++++++++++++++++++++- 3 files changed, 36 insertions(+), 46 deletions(-)

2 1

[PATCH 06/10] win32u: Move DPI mapping from GetMonitorInfoW.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/sysparams.c | 17 +---------------- dlls/win32u/sysparams.c | 38 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 39 insertions(+), 16 deletions(-)

2 1

[PATCH 05/10] win32u: Move NtUserGetDpiForMonitor implementation from user32.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/sysparams.c | 24 ---------------- dlls/user32/user32.spec | 2 +- dlls/win32u/syscall.c | 1 + dlls/win32u/sysparams.c | 61 +++++++++++++++++++++++++++++++++++++++++ dlls/win32u/win32u.spec | 2 +- dlls/wow64win/syscall.h | 1 + dlls/wow64win/user.c | 10 +++++++ include/ntuser.h | 1 + 8 files changed, 76 insertions(+), 26 deletions(-)

2 1

[PATCH 04/10] win32u: Implement NtUserGetSystemDpiForProcess.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

2 1

[PATCH 03/10] win32u: Implement NtUserGetProcessDpiAwarenessContext and NtUserSetProcessDpiAwarenessContext.
by Jacek Caban Dec. 6, 2021

Dec. 6, 2021

And use them in user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/sysparams.c | 45 ++++++++++++++++++++--------------------- dlls/win32u/syscall.c | 2 ++ dlls/win32u/sysparams.c | 38 ++++++++++++++++++++++++++++++++++ dlls/win32u/win32u.spec | 4 ++-- dlls/wow64win/syscall.h | 2 ++ dlls/wow64win/user.c | 15 ++++++++++++++ include/ntuser.h | 9 +++++++++ 7 files changed, 90 insertions(+), 25 deletions(-)

2 1