Closed Bug 1699276 Opened 6 months ago Closed 6 months ago

Remove JSID_IS_ATOM from js/src/debugger

Categories

(Core :: JavaScript Engine, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: mgaudet, Assigned: cOde.A4sh, Mentored)

References

Details

(Keywords: good-first-bug)

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #1699273 +++

Note: this bug is currently reserved for Outreachy applicants for the Spring/Summer 2021 cycle. If it has not been completed by the end of the application cycle, we will open it up.

Background

The JSID_IS_ATOM calls reflect an old style of SpiderMonkey programming that we think we can start to get away from, internally to the engine.

This Bug

This bug is about removing uses of JSID_IS_ATOM within the js/src/debugger files. Instead of using JSID_IS_ATOM we should be using JS::PropertyKey::isAtom, which is actually how JSID_IS_ATOM is implemented.

Prerequisites

Before getting started, you'll want to

How you’ll know you’re done

  • The use of JSID_IS_ATOM is removed
  • SpiderMonkey builds
  • The tests run successfully.

Getting Help

Feel free to leave comments on this bug for questions, or, if you have more synchronous questions about this bug, feel free to drop into the #spidermonkey channel on chat.mozilla.org.

Tips:

  • Not sure if the code you've been editing is getting run? Insert a call to MOZ_CRASH, a macro which will crash when executed, and run the entire test suite with an optimized build (for speed). If you see crashes, you can then use a debug build to make sure it's crashing in your code!

I am an outreachy applicant.I want to work on this bug.

i passed all the test for the updated code .

Assignee: nobody → cOde.A4sh
Status: NEW → ASSIGNED
Attachment #9213523 - Attachment description: Bug 1699276-uses of JSID_IS_ATOM are replaced with JS::PropertyKey::isAtom.r=mgaudet → Bug 1699276: Uses of JSID_IS_ATOM are replaced with JS::PropertyKey::isAtom r=mgaudet
Attachment #9213572 - Attachment is obsolete: true
Pushed by mgaudet@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af733fba4ae4
Uses of JSID_IS_ATOM are replaced with JS::PropertyKey::isAtom r=mgaudet
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.