Fixes the ‘auto_ptr’ is not a member of ‘std’ #5

Closed
wants to merge 7 commits into
from

Projects

None yet

2 participants

@SaltwaterC

There's a bunch of errors since 0.1.2 when building gzbz2:

[1/2] cxx: compress.cc -> build/default/compress_1.o
../compress.cc: In static member function ‘static v8::Handle Gzip::GzipDeflate(const v8::Arguments&)’:
../compress.cc:193: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:193: error: expected primary-expression before ‘>’ token
../compress.cc:193: error: ‘bw’ was not declared in this scope
../compress.cc:210: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:210: error: expected primary-expression before ‘>’ token
../compress.cc: In static member function ‘static v8::Handle Gunzip::GunzipInflate(const v8::Arguments&)’:
../compress.cc:405: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:405: error: expected primary-expression before ‘>’ token
../compress.cc:405: error: ‘bw’ was not declared in this scope
../compress.cc:422: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:422: error: expected primary-expression before ‘>’ token
../compress.cc: In static member function ‘static v8::Handle Bzip::BzipDeflate(const v8::Arguments&)’:
../compress.cc:629: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:629: error: expected primary-expression before ‘>’ token
../compress.cc:629: error: ‘bw’ was not declared in this scope
../compress.cc:646: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:646: error: expected primary-expression before ‘>’ token
../compress.cc: In static member function ‘static v8::Handle Bunzip::BunzipInflate(const v8::Arguments&)’:
../compress.cc:843: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:843: error: expected primary-expression before ‘>’ token
../compress.cc:843: error: ‘bw’ was not declared in this scope
../compress.cc:860: error: ‘auto_ptr’ is not a member of ‘std’
../compress.cc:860: error: expected primary-expression before ‘>’ token

This one line patch makes it go away.

Owner
Woodya commented Oct 26, 2011

create a pull request just for c9f7941 (include memory) and i'll accept it. otherwise it'll add lots of extraneous commits.

@Woodya Woodya closed this Oct 26, 2011

Don't know how. Aka my git-rebase fu is not strong. Can you add

#include <memory>

at the top of compress.cc? Besides the bunch of commits, that's the actual diff between my master branch and yours.

Owner
Woodya commented Oct 28, 2011

ok. i'm going to see if i can learn enough git to cherry-pick your commit.
otherwise i'll just add the include manually. thanks for fixing this.
i build on osx and it worked, you building on a linux variant?
-w

On Thu, Oct 27, 2011 at 12:22 AM, Ștefan Rusu <
reply@reply.github.com>wrote:

Don't know how. Aka my git-rebase fu is not strong. Can you add

#include <memory>

at the top of compress.cc? Besides the bunch of commits, that's the actual
diff between my master branch and yours.

Reply to this email directly or view it on GitHub:
#5 (comment)

Owner
Woodya commented Oct 28, 2011

ok, cherry picked: fyi i used this git cherrypick tutorial
http://mike.bailey.net.au/2008/08/cherry-picking-git-commits/

Yes, I tried to build this on both Ubuntu 10.04 and 10.10. Both failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment