Remove jsstr.h and vm/String.h from INSTALLED_HEADERS

RESOLVED FIXED in mozilla13

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: billm, Assigned: luke)

Tracking

unspecified
mozilla13
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Any takers?
(Assignee)

Comment 1

6 years ago
I can.  Does this depend on jscntxt.h removal?
(Reporter)

Comment 2

6 years ago
(In reply to Luke Wagner [:luke] from comment #1)
> I can.  Does this depend on jscntxt.h removal?

I don't think so.
Blocks: 554088
(Assignee)

Comment 3

6 years ago
Created attachment 592560 [details] [diff] [review]
un-install vm/String.h and some others

The patch also kills some random installed headers that were easy: jsautokw.h, jsclone.h, jsopcode.tbl, jsopcode.h, jsstdint.h.
Assignee: general → luke
Status: NEW → ASSIGNED
Attachment #592560 - Flags: review?(wmccloskey)
(Assignee)

Comment 4

6 years ago
Green on try
(Reporter)

Comment 5

6 years ago
Comment on attachment 592560 [details] [diff] [review]
un-install vm/String.h and some others

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

::: js/src/jsatominlines.h
@@ +198,5 @@
>  }
>  
> +inline
> +AtomHasher::Lookup::Lookup(const JSAtom *atom)
> + : chars(atom->chars()), length(atom->length()), atom(atom)

I think we usually have two spaces before the : here.
Attachment #592560 - Flags: review?(wmccloskey) → review+
(Assignee)

Comment 6

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f5ff7098d6a4
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/f5ff7098d6a4
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.