AudioNet / node-core-audio

Bindings for PortAudio giving JavaScript access to sound card samples (mostly unmaintained)
MIT License
330 stars 66 forks source link

OS X 10.8 (Mtn. Lion) NPM compile warnings. #14

Closed Fishrock123 closed 10 years ago

Fishrock123 commented 11 years ago

I'm getting these compile warnings, just a heads up.

> node-gyp rebuild

  CXX(target) Release/obj.target/NodeCoreAudio/NodeCoreAudio/AudioEngine.o
../NodeCoreAudio/AudioEngine.cpp:255:4: warning: 'auto' type specifier is a C++11 extension
      [-Wc++11-extensions]
                        auto tempBuffer = Local<Array>( Array::New(m_uSamplesPerBuffer) );
                        ^
In file included from ../NodeCoreAudio/AudioEngine.cpp:9:
/Users/Jeremiah/Documents/node-voicechat/node_modules/node-core-audio/NodeCoreAudio/AudioEngine.h:80:6: warning: 
      private field 'jsAudioThread' is not used [-Wunused-private-field]
                                        jsAudioThread;                  //!< Our JavaScript ...
                                        ^
2 warnings generated.
  CXX(target) Release/obj.target/NodeCoreAudio/NodeCoreAudio/NodeCoreAudio.o
  SOLINK_MODULE(target) Release/NodeCoreAudio.node
  SOLINK_MODULE(target) Release/NodeCoreAudio.node: Finished
ZECTBynmo commented 11 years ago

It looks like you're compiling without C++0x enabled. There was another issue where someone was able to compile by adding a compile flag to the gyp file, let me find that quickly.

Fishrock123 commented 11 years ago

You mean #13?

'cflags_cc': [ '-std=c++0x' ] Is still in the gyp.

ZECTBynmo commented 11 years ago

Oh I just realized you said they were warnings. Thanks for the heads up, not sure why that's worth raising a warning about.

Yes, that's the issue I was referring to.

ZECTBynmo commented 10 years ago

I think I remember this issue being resolved. Am I right @Fishrock123? Closing for now, but feel free to reopen

Fishrock123 commented 10 years ago

Nope. Here's a gist: https://gist.github.com/Fishrock123/8155792

Same as in the original issue comment.