Esme Povirk (@madewokherd) commented about dlls/sane.ds/ui.c:
{SetDlgItemTextA(hwnd, IDC_MANUFACTURER , activeDS.identity.Manufacturer);SetDlgItemTextA(hwnd, IDC_PRODUCTFAMILY, activeDS.identity.ProductFamily);SetDlgItemTextA(hwnd, IDC_PRODUCTNAME , activeDS.identity.ProductName);SetDlgItemInt(hwnd, IDC_PAGE, activeDS.scannedImages+1, TRUE);}break;case WM_COMMAND:{switch (LOWORD(wParam)){case IDCANCEL:WARN("User cancelled\n");activeDS.userCancelled = TRUE;}}
Cases should end with a `break;` even if they're the last one.