Bug 148611

Summary:

JSC property attributes should fit in a byte

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

barraclough, basile_clement, benjamin, commit-queue, ggaren, mark.lam, mhahnenb, msaboff, nrotem, oliver, saam, sam

Priority:

Version:

Other

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

148610

Attachments:

Description	Flags
the patch	fpizlo: review-
a different approach	none
the patch	none
the patch	sam: review+

Description Filip Pizlo 2015-08-29 13:14:32 PDT

Patch forthcoming.

Comment 1 Filip Pizlo 2015-08-29 13:21:46 PDT

Created attachment 260226 [details]
the patch

Comment 2 Filip Pizlo 2015-08-29 13:38:04 PDT

Comment on attachment 260226 [details]
the patch

Nope, this doesn't work.

Comment 3 Filip Pizlo 2015-08-29 21:02:08 PDT

Created attachment 260232 [details]
a different approach

Not yet ready for review though.  Just seeing if EWS is happy.

Comment 4 WebKit Commit Bot 2015-08-29 21:03:55 PDT

Attachment 260232 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:43:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:47:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 2 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.

Comment 5 Filip Pizlo 2015-08-29 21:59:18 PDT

Created attachment 260236 [details]
the patch

Comment 6 WebKit Commit Bot 2015-08-29 22:01:29 PDT

Attachment 260236 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:43:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:47:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:48:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 3 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.

Comment 7 Filip Pizlo 2015-08-29 22:38:23 PDT

Created attachment 260237 [details]
the patch

Comment 8 WebKit Commit Bot 2015-08-29 22:40:47 PDT

Attachment 260237 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:43:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:47:  One space before end of line comments  [whitespace/comments] [5]
ERROR: Source/JavaScriptCore/runtime/PropertySlot.h:48:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 3 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.

Comment 9 Sam Weinig 2015-08-30 10:26:54 PDT

Is there any reason to keep the static hashtable specific attributes in the Attribute enum anymore? Should we instead have HashTableValue have two fields, m_attributes (of type Attributes) which contains the JSObject attributes and a new m_staticAttributes (awful name) which has the static hashtable specific values?

Comment 10 Filip Pizlo 2015-08-30 10:36:30 PDT

(In reply to comment #9)
> Is there any reason to keep the static hashtable specific attributes in the
> Attribute enum anymore? Should we instead have HashTableValue have two
> fields, m_attributes (of type Attributes) which contains the JSObject
> attributes and a new m_staticAttributes (awful name) which has the static
> hashtable specific values?

I think that might be good follow-up change. The benefit of my approach is that it is a much smaller change, since I don't have to touch the code generators that use the static hash table attributed.

Comment 11 Filip Pizlo 2015-08-30 15:34:34 PDT

Landed in http://trac.webkit.org/changeset/189160