Andrew Talbot wrote: > if (fodInfos->ofnInfos->Flags & OFN_ALLOWMULTISELECT) > size += 1; > /* return needed size in first two bytes of lpstrFile */ > - *(WORD *)fodInfos->ofnInfos->lpstrFile = size; > + *fodInfos->ofnInfos->lpstrFile = size; > FILEDLG95_Clean(hwnd); > ret = EndDialog(hwnd, FALSE); > COMDLG32_SetCommDlgExtendedError(FNERR_BUFFERTOOSMALL); > @@ -763,7 +763,7 @@ static BOOL PRINTDLG_SetUpPaperComboBoxW(HWND hDlg, > NrOfEntries = DeviceCapabilitiesW(PrinterName, PortName, > fwCapability_Names, Names, dm); > NrOfEntries = DeviceCapabilitiesW(PrinterName, PortName, > - fwCapability_Words, (LPWSTR)Words, dm); > + fwCapability_Words, Words, dm); > > /* reset any current content in the combobox */ > SendDlgItemMessageW(hDlg, nIDComboBox, CB_RESETCONTENT, 0, 0); >
I think these two casts should stay as LPWSTR is compatible with WORD * only by coincidence - one is intended to hold characters and the other numbers. -- Rob Shearman
