Closed Bug 1444169 Opened 2 years ago Closed 2 years ago

Clean up all MinGw capitalization issues

Categories

(Firefox :: General, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: tjr, Assigned: tjr)

References

Details

Attachments

(1 file)

If we're going to land Bug 1444167, there will be a few capitalization issues that will need to be fixed before-hand. This bug is for fixing them.
Attachment #8957272 - Attachment is obsolete: true
Attachment #8957272 - Flags: review?(ahalberstadt)
Comment on attachment 8957272 [details]
Bug 1444169 Fix capitalization for a few Windows files

https://reviewboard.mozilla.org/r/226194/#review232194

::: gfx/layers/d3d11/CompositorD3D11.cpp:38
(Diff revision 2)
>  
>  #include "D3D11ShareHandleImage.h"
>  #include "DeviceAttachmentsD3D11.h"
>  
> -#ifdef __MINGW32__
>  #include <versionhelpers.h> // For IsWindows8OrGreater

We should just be using our version testing code in MFBT here, but that's not your problem.
Attachment #8957272 - Flags: review?(aklotz) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/93b6538bc167
Fix capitalization for a few Windows files r=aklotz
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/93b6538bc167
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.