Re: [PATCH 0/5] MR5865: win32u: Move some invisible surface conditions out of the drivers.

[Emil Velikov (＠xexaxo)]

Emil Velikov (@xexaxo) commented about dlls/winex11.drv/window.c:
> }
>
> -static inline BOOL get_surface_rect( const RECT *visible_rect, RECT *surface_rect )
> -{
> - *surface_rect = NtUserGetVirtualScreenRect();
> -
> - if (!intersect_rect( surface_rect, surface_rect, visible_rect )) return FALSE;
> - OffsetRect( surface_rect, -visible_rect->left, -visible_rect->top );
> - surface_rect->left &= ~31;
> - surface_rect->top  &= ~31;
> - surface_rect->right  = max( surface_rect->left + 32, (surface_rect->right + 31) & ~31 );
> - surface_rect->bottom = max( surface_rect->top + 32, (surface_rect->bottom + 31) & ~31 );
> - return TRUE;

Would it make sense to add a comment where these alignment numbers come from - they seem to vary across platforms, sort of.

Again not something for this MR, just noticed while browsing.