ensure mozilla/mscom/Utils.h is included with uniform casing

RESOLVED FIXED in Firefox 51

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
We install mozilla/mscom/Utils.h, but include it with an all-lowercased
path in some places.  clang-cl warns about this, and even though this is
Windows-only code and will therefore work, it's better to make
everything consistent.
(Assignee)

Comment 1

2 years ago
Created attachment 8786033 [details] [diff] [review]
ensure mozilla/mscom/Utils.h is included with uniform casing
Attachment #8786033 - Flags: review?(aklotz)
Comment on attachment 8786033 [details] [diff] [review]
ensure mozilla/mscom/Utils.h is included with uniform casing

Review of attachment 8786033 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Attachment #8786033 - Flags: review?(aklotz) → review+

Comment 3

2 years ago
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/65d3f6171572
ensure mozilla/mscom/Utils.h is included with uniform casing; r=aklotz

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/65d3f6171572
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.