790781 - Hal.cpp:409: error: ‘sSystemTimeObserver’ was not declared in this scope

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Ameya Bapat]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.1 (KHTML, like Gecko) Chrome/21.0.1180.89 Safari/537.1

Steps to reproduce:

I did hg pull -u to get  latest changes in mozilla-central directory. i tried to compile code by make -f client.mk on linux ubuntu platform.


Actual results:

I got following errors during compilation: The respective file seems to be modified in recent time.
Errors
1.  /mozilla-central/hal/Hal.cpp:409: error: ‘sSystemTimeObserver’ was not declared in this scope

It could be because of template decalration failed. Please suggest necessary patch or repair to it.

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

This is using C++10 syntax. Do we require a C++10 compiler now?

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

(er, C++11, of course)

Comment 3

[Jesse Ruderman]

I'm guessing this is the same problem:

/Users/jruderman/mozilla-central/hal/Hal.cpp:409:51: error: a space is required
      between consecutive right angle brackets (use '> >')
static StaticAutoPtr<ObserverList<SystemTimeChange>> sSystemTimeObserver;
                                                  ^~
                                                  > >

/usr/bin/clang++ --version
Apple clang version 3.0 (tags/Apple/clang-211.12) (based on LLVM 3.0svn)
Target: x86_64-apple-darwin11.4.0
Thread model: posix

Ameya, does https://hg.mozilla.org/mozilla-central/rev/657097e4f430 fix it for you?

Comment 4

[Ameya Bapat]

(In reply to Jesse Ruderman from comment #3)

> Ameya, does https://hg.mozilla.org/mozilla-central/rev/657097e4f430 fix it
> for you?
Yes... It seems '<<' to '< <' change is working. Thanks..