Closed Bug 894181 Opened 11 years ago Closed 11 years ago

Convert a bunch of SHAPE_* macros to inline functions to eliminate warnings, enhance debuggability

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

      No description provided.
Attached patch PatchSplinter Review
This eliminates another batch of clang+ccache warnings.
Attachment #776114 - Flags: review?(terrence)
Comment on attachment 776114 [details] [diff] [review]
Patch

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

Maybe move these to sanely named members of Shape as a followup?

r=me

::: js/src/vm/Shape.h
@@ -1044,4 @@
>          SkipRoot skip;
>          MOZ_DECL_USE_GUARD_OBJECT_NOTIFIER
>      };
> - };

Heh, wonder when that snuck in.
Attachment #776114 - Flags: review?(terrence) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: