199366 - OpenVMS compiler craps out on "readonly" label

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Colin Blake]

OpenVMS doesn't like something to be named "readonly". Its a reserved word in
the OpenVMS compiler. Yeah, I know, it shouldn't be, and I've complained, but it
still ain't fixed.

Recent checkin introduced a label named "readonly". Fix is to rename it to
"read_only". Patch coming.

Comment 1

[Colin Blake]

Attachment: Rename readonly read_only

Comment 2

[Colin Blake]

Asking for blocking1.4a status as this breaks the OpenVMS build.

Roger, can you give me an r= and suggest appropriate sr= (Brendan?).

Comment 3

[Brendan Eich [:brendan]]

js/src doesn't require sr=.  I will fix this today -- no-brainer.

Colin, is readonly reserved in OpenVMS's C++ compiler, too?

/be

Comment 4

[Brendan Eich [:brendan]]

Oops, a little fast on the draw there.

/be

Comment 5

[Colin Blake]

> js/src doesn't require sr=.  I will fix this today -- no-brainer.

Thanks Brendan.

> Colin, is readonly reserved in OpenVMS's C++ compiler, too?

Nope. The C++ compiler is happy with readonly as a label.

Comment 6

[Colin Blake]

Almost three years to the day since this one first bit me :-)
http://www.geocrawler.com/mail/msg.php3?msg_id=3531296&list=137

Comment 7

[Brendan Eich [:brendan]]

Fixed.

/be

Comment 8

[Phil Schwartau]

Checkin verified -