チケット #41071

Add publicly known LCS_sRGB and LCS_WINDOWS_COLOR_SPACE

登録: 2020-12-24 06:52 最終更新: 2021-04-12 17:39

報告者:
担当者:
チケットの種類:
状況:
完了
コンポーネント:
(未割り当て)
マイルストーン:
(未割り当て)
優先度:
5 - 中
重要度:
5 - 中
解決法:
受領
ファイル:
1
投票
点数: 0
No votes
0.0% (0/0)
0.0% (0/0)

詳細

Here is a public source to add definitions of LCS_sRGB and LCS_WINDOWS_COLOR_SPACE

https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/eb4bbd50-b3ce-4917-895c-be31f214797f

Also here is public definitions of PROFILE_LINKED and PROFILE_EMBEDDED with and without LCS_ prefix

https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/3c289fe1-c42e-42f6-b125-4b5fc49a2b20

Please add to wingdi.h

チケットの履歴 (17 件中 3 件表示)

2020-12-24 06:52 更新者: apodtele
  • 新しいチケット "Add publicly known LCS_sRGB and LCS_WINDOWS_COLOR_SPACE " が作成されました
2020-12-29 07:52 更新者: keith
  • チケットの種類Issues から Feature Request に更新されました
コメント

Thanks for these references. Please note that this should have been filed as a "feature request", not as an "issues" report; I've redesignated it accordingly.

Do you happen to know, for sure, what versions of Windows first supported these? If not, we'll have to accept the guidance of the "footnotes", which suggest that the first pair first appeared in Win95/WinNT4, while the latter pair didn't appear until Win98/Win2K.

2020-12-29 09:59 更新者: apodtele
コメント

I only know these must be defined as possible values of bV5CSType in BIITMAPV5HEADER (ca Win98) https://docs.microsoft.com/en-us/windows/win32/api/wingdi/ns-wingdi-bitmapv5header

However, VC2017 did not complain when I used LCS_sRGB in BITMAPV4HEADER (ca Win95).

2021-01-03 02:50 更新者: apodtele
コメント

To complete BITMAPV5HEADER constants, also add LCS_GM_ABS_COLORIMETRIC from https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/9fec0834-607d-427d-abd5-ab240fb0db38

There is also BI_CMYK, BI_CMYKRLE8, BI_CMYKRLE4 in https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/4e588f70-bd92-4a6f-b77f-35d0feaf7a57 and BI_ALPHABITFIELDS slightly off spec but searchable.

2021-02-07 02:58 更新者: apodtele
コメント

How can I possibly help to add the missing definitions? Would you like me to prepare a patch? I would like MinGW to support the BMP feature properly.

2021-02-07 03:22 更新者: keith
コメント

Reply To apodtele

How can I possibly help to add the missing definitions? Would you like me to prepare a patch?

A patch would certainly be welcome, (and helpful), although this is on my to-do list for the next release, in any event. If you do elect to submit a patch, please include a ChangeLog entry, (in the patch header ... not in diff format), and indicate how you would like the attribution to appear, in the ChangeLog file itself.

I would like MinGW to support the BMP feature properly.

As would I. Thanks for your interest, and support.

2021-02-08 03:28 更新者: apodtele
  • 添付ファイル wingdi.diff (File ID: 5993) が付加されました
2021-02-08 03:32 更新者: apodtele
コメント

A diff is attached.

I also found multiple discrepancies with this block https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/55961a7b-f6f6-4f0b-a76b-b4cf652bc86b but I am not sure if you want to correct or just add them.

2021-02-08 06:26 更新者: keith
  • 添付ファイル wingdi.diff (File ID: 5993) が削除されました
2021-02-08 07:15 更新者: keith
  • 担当者(未割り当て) から keith に更新されました
コメント

Reply To apodtele

A diff is attached.

Thanks. That is helpful ... although you did do exactly what I asked you not to, w.r.t. the ChangeLog entry, so I've taken the liberty of correcting it for you. I've also made some minor layout adjustments, so that this appears correct in the patch, as well as in the <wingdi.h> file itself.

I also found multiple discrepancies with this block https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-wmf/55961a7b-f6f6-4f0b-a76b-b4cf652bc86b but I am not sure if you want to correct or just add them.

If anything else is missing, then please add it; if it's already present, but incorrect, then please correct it.

Perhaps you could also clarify something for me: in the Microsoft docs, to which you referred me, I see no reference to PROFILE_LINKED, or to PROFILE_EMBEDDED, as aliases for LCS_PROFILE_LINKED, and LCS_PROFILE_EMBEDDED respectively. Whilst I don't mind leaving the aliases in place, can you please provide alternative references to justify doing so?

2021-02-08 11:29 更新者: apodtele
コメント

Re. PROFILE_* vs LCS_PROFILE_*

I just see both used in MS docs: BITMAPV5HEADER and BitmapV5Header Object. Go figure.

2021-02-14 01:57 更新者: keith
コメント

Reply To apodtele

Re. PROFILE_* vs LCS_PROFILE_*
I just see both used in MS docs: BITMAPV5HEADER and BitmapV5Header Object. Go figure.

Ambiguous, indeed. I guess it does make sense to support both forms, although officially, the LCS_prefixed form should be preferred.

Anyway, I committed this. Is there anything else you would like to add, within the scope of this ticket, or may I close it?

2021-02-14 02:25 更新者: apodtele
コメント

Thank you. Please close it.

2021-02-14 02:53 更新者: keith
  • 状況オープン から 完了 に更新されました
  • 解決法なし から 受領 に更新されました
2021-04-12 00:21 更新者: apodtele
コメント

It would be prudent to release updated WSL before this summer. Many students use the break to try new tools or participate in GSoC.

2021-04-12 17:39 更新者: keith
コメント

Reply To apodtele

It would be prudent to release updated WSL before this summer ...

Hmm, yes ... it was on my TODO list, and I guess it was overdue. Done now: package files published yesterday, and mingw-get catalogue updated today. New release designated as composite of mingwrt-5.4.2, and w32api-5.4.2.

添付ファイルリスト

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする