C++ Wrapper on hiredis

[kappe]

hi salvatore,

i'm working on a c++ wrapper class upon hiredis (using stl and boost).
i had to make some typecast in the headerfiles of hiredis and leave some compiler options
(new commandline: gcc -fPIC -Wall -Wwrite-strings -O2 -c ...)
do you think this could become problematic?

regards

kappe

[antirez]

Hello,

it seems ok, our goal is that this should compile out of the box on
Linux, *BSD, and Mac OS X.

Cheers,
Salvatore

[pietern]

Can you elaborate on the compiler options that should be left out? The only difference with the options you provide is that the current options have a couple extra warning-related options enabled.

I've just added the commit rtecco/hiredis@8b8aec7b to allow C++ programs to include and use hiredis.

[kappe]

Hi pietern,

I've build a git repository for my C++ wrapper class on hiredis. I've "patched" hiredis to successfully compile with g++.
See README on https://github.com/kappe/ccredis

Reguards

kappe

[pietern]

If you can outline the changes that are needed (e.g. provide a diff somewhere), then I can check if it is feasible to make changes to the main repo so it compiles with g++. Closing this issue now, because it is no longer an issue.

[kappe]

I used c++ casts like static_cast<char*>( ).
You can't use in c. So it makes no sense to provide an patch.