No content change: biSizeImage docs for BI_RLE8/BI_RLE4 live in sdk-api, not this repo - #6106
Draft
John Kennedy (GrantMeStrength) with Copilot wants to merge 1 commit into
Draft
No content change: biSizeImage docs for BI_RLE8/BI_RLE4 live in sdk-api, not this repo#6106John Kennedy (GrantMeStrength) with Copilot wants to merge 1 commit into
John Kennedy (GrantMeStrength) with Copilot wants to merge 1 commit into
Conversation
Contributor
|
Learn Build status updates of commit 4d94c95:
|
Copilot
AI
changed the title
[WIP] Clarify biSizeImage for BI_RLE8 and BI_RLE4 bitmaps
No content change: biSizeImage docs for BI_RLE8/BI_RLE4 live in sdk-api, not this repo
Aug 31, 2026
#John wick 02 (jacobmerlincornelius02-alt)
left a comment
There was a problem hiding this comment.
Doc
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue #5265 asks for clarification of
biSizeImage(andbV4SizeImage/bV5SizeImage) whenbiCompressionisBI_RLE8orBI_RLE4: compressed size, uncompressed size, or either, and what happens when the value exceeds the data present in the file.This PR contains no file changes. The pages in question are not in this repository.
Finding
BITMAPINFO,biSizeImage,BI_RLE8, andBI_RLE4returns zero matches across all 9,282 tracked files.BITMAPINFOHEADER/BITMAPV4HEADER/BITMAPV5HEADER(/windows/win32/api/wingdi/…) are authored inMicrosoftDocs/sdk-api; bitmap storage and compression conceptual topics (/windows/win32/gdi/…) are in the Win32 conceptual repo. Neither is reachable from this branch.hub/apps/would duplicate content this repo doesn't own, so no page was created.Suggested next step
Transfer #5265 to
MicrosoftDocs/sdk-api(wingdi.h) and apply the wording below to the three size members.This parallels the existing
BI_JPEG/BI_PNGsentence already on the page.Open questions for a GDI owner
Not answered in the proposed wording, and not safe to guess — the reporter is writing a BMP parser and needs to know whether to accept or reject these files:
0x00 0x01) if fewer thanbiSizeImagebytes have been consumed?biSizeImageexceeds the data actually present, is the bitmap rejected, or is the available data decoded and the remainder ignored?Separate issue on the same page
Earlier responses on #5265 pointed to Media Foundation and
MFCalculateBitmapImageSize. That doesn't apply here — these structures are current for GDI device-independent bitmaps, and the deprecation notice atopBITMAPINFOHEADERrefers to the associated media feature rather than the structure. Worth correcting in the samesdk-apipass.