search

MicrosoftDocs / feedback

📢 docs.microsoft.com site feedback
https://learn.microsoft.com
Creative Commons Attribution 4.0 International
239 stars 160 forks source link

Mismatch between docs and ECMA on PE header #942

Open Pzixel opened 5 years ago

Pzixel commented 5 years ago

Link to article: https://docs.microsoft.com/en-us/windows/desktop/Debug/pe-format#optional-header-data-directories-image-only

Problem: The problem is that documentation names the second last field CLR Runtime Header

However, if you look on ECMA335 which actually introduced this field it names it CLI Header (see link, page 280-281).

I have a discussion in one project about how to name this field, and I'd like to have some consistent naming here. I'm up to using ECMA335 naming, but other guys think that documentation is more correct. Either way, it should be consistent across the documentation, so either should be ECMA updated (since C# 8.0 is coming, it won't be difficult to fix this if it's an error), or rather update this page (which I find more correct).

welcome[bot] commented 5 years ago

Thank you for creating the issue! One of our team members will get back to you shortly with additional information.

ginami commented 5 years ago

Hi @Pzixel -- thank you for your feedback! We'll have someone take a look and get back to you on this.