Closed Bug 302264 Opened 19 years ago Closed 19 years ago

fix soon-to-be-fatal warning: bad access to protected variable

Categories

(Camino Graveyard :: General, defect)

PowerPC
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jaas, Assigned: jaas)

Details

Attachments

(1 file)

BrowserWindowController.mm: In function 'int TabBarVisiblePrefChangedCallback(const char*, void*)':
BrowserWindowController.mm:3503: warning: instance variable 'mTabBrowser' is @protected; this will be a 
hard error in the future
Attached patch fix v1.0 β€” β€” Splinter Review
This got rid of the warning, I'm on x86 now and can test to see if the patch
really works on PPC soon. Not requesting review until I test, but its easy so
feel free to review if you want.
Assignee: pinkerton → joshmoz
Status: NEW → ASSIGNED
Of course, we'd actually delete the bad replaced line instead of commenting it out if we were going to land 
that patch :)
Comment on attachment 190631 [details] [diff] [review]
fix v1.0

r=mark, why not just whack the commented line?
Attachment #190631 - Flags: review+
landed - I guess mark just didn't see my comment right before his :)
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: