474365 - OS/2 breaks: follow-up to Bug 461047 - Replace nsStringArray with nsTArray<nsString>

Status: RESOLVED FIXED

(Core :: Widget, defect)

Description

[Walter Meinl]

User-Agent:       Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.9.2a1pre) Gecko/20090120 Minefield/3.2a1pre
Build Identifier: 

After checkin for bug461047, OS/2 build breaks in widget/src/os2/nsDeviceContextSpecOS2.cpp
E:/usr/src/hg/mozilla-central/widget/src/os2/nsDeviceContextSpecOS2.cpp: In 
   member function `nsString* GlobalPrinters::GetStringAt(int)':
E:/usr/src/hg/mozilla-central/widget/src/os2/nsDeviceContextSpecOS2.cpp:86: error: `
   StringAt' undeclared (first use this function)
E:/usr/src/hg/mozilla-central/widget/src/os2/nsDeviceContextSpecOS2.cpp:86: error: (Each
   undeclared identifier is reported only once for each function it appears 
   in.)
make.exe[6]: *** [nsDeviceContextSpecOS2.o] Error 1


Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Walter Meinl]

Attachment: patch - s/StringAt/ElementAt (like in gtk)

Peter, this patch includes the same exchange also in the corresponding BeOS file, cause the BeOS build would probably break, too. Do you know a BeOS guy to cc?

Comment 2

[Peter Weilbacher]

Comment on attachment 357740 [details] [diff] [review]
patch - s/StringAt/ElementAt (like in gtk)

Looks good.

Comment 3

[Peter Weilbacher]

I guess that for mozilla-central BeOS is so broken at this point that this one fix doesn't make a difference. On the other hand I think that the BeOS guys are happy, if we fix it for them. So I am going to push this now.

Comment 4

[Peter Weilbacher]

http://hg.mozilla.org/mozilla-central/rev/49e3f6fb8076