1199887 - Various renaming, code cleanups to do before bug 1187234/bug 1179003

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jeff Walden [:Waldo]]

This is mostly some dumb stuff noticed around is-class assertions that I removed in bug 1179003.  A little grab-baggy, but pretty minimal.

(Given the size of changes in bug 1179003 and bug 1187234, and my understanding of your school schedule, I held off on requesting review from you, evilpie.  If you want to look/review anyway, go ahead.)

Comment 1

[Jeff Walden [:Waldo]]

Attachment: Rename StringRegExpGuard::init(JSContext*, JSObject*) to initRegExp

One of the callers checks class directly just prior to calling this.  The other does a proxy-permitting check a little earlier.  Too bad we can't use Handle<RegExpObject*> or something to connote this class-checked status -- at least not with the existing APIs.  Maybe we could invent a template class to convey (X or wrapped X) somehow.  But definitely another bug, if it ever happens.

Comment 2

[Jeff Walden [:Waldo]]

Attachment: Make str_replace_regexp_raw return a JSString*, rather than return its always-string result via outparam

The interpreter-based path where returning as Value makes sense, isn't the primary sense any more.  Return the string that everyone *but* the interpreter wants, directly.

The last patch here does likewise for str_replace_string_raw -- just keeping them separate for smallness.

Comment 3

[Jeff Walden [:Waldo]]

Attachment: Make str_replace_string_raw return a JSString*, rather than return its always-string result via outparam

Comment 4

[Tom S. (please needinfo tschuster)]

Comment on attachment 8654465 [details] [diff] [review]
Rename StringRegExpGuard::init(JSContext*, JSObject*) to initRegExp

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

::: js/src/jsstr.cpp
@@ +2122,5 @@
>          obj_ = regexp;
>  
>          MOZ_ASSERT(ObjectClassIs(obj_, ESClass_RegExp, cx));
>  
>          if (!RegExpToShared(cx, obj_, &re_))

Can you fix this?

Comment 5

[Tom S. (please needinfo tschuster)]

Comment on attachment 8654466 [details] [diff] [review]
Make str_replace_regexp_raw return a JSString*, rather than return its always-string result via outparam

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

Very nice.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/dde65be15e24
https://hg.mozilla.org/integration/mozilla-inbound/rev/9de14ef7802e
https://hg.mozilla.org/integration/mozilla-inbound/rev/e0d198f50e59

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/dde65be15e24
https://hg.mozilla.org/mozilla-central/rev/9de14ef7802e
https://hg.mozilla.org/mozilla-central/rev/e0d198f50e59