Skip to content

Conversation

laanwj
Copy link
Member

@laanwj laanwj commented Jun 21, 2015

Now that boost spirit is no longer used, -DBOOST_SPIRIT_THREADSAFE doesn't need to be passed to the compiler anymore.

@jgarzik
Copy link
Contributor

jgarzik commented Jun 21, 2015

ACK

FYI src/rpcprotocol.cpp has a mention of json-spirit in a comment (a URL) also

@jonasschnelli
Copy link
Contributor

ACK.
Agree with jgarzik on the removal of rpcprotocol.cpp L254 in this PR.

@Diapolo
Copy link

Diapolo commented Jun 21, 2015

There also is a comment in univalue.h, perhaps related?

// The following were added for compatibility with json_spirit.
// Most duplicate other methods, and should be removed.

@fanquake
Copy link
Member

utACK

@jonasschnelli
Copy link
Contributor

@Diapolo: right. UniValue has some json_spirit compatibility functions which should be removed/changed (but not in this PR). Also some functions should be renamed IsNull() vs get_str() looks after a json_spirit compatibility relict.

Now that boost spirit is no longer used, `-DBOOST_SPIRIT_THREADSAFE`
doesn't need to be passed to the compiler anymore.
@laanwj laanwj force-pushed the 2015_06_smells_like_boost_spirit branch from ce6ebe3 to b666820 Compare June 21, 2015 12:09
@paveljanik
Copy link
Contributor

ACK

1 similar comment
@theuni
Copy link
Member

theuni commented Jun 22, 2015

ACK

@laanwj laanwj merged commit b666820 into bitcoin:master Jun 22, 2015
laanwj added a commit that referenced this pull request Jun 22, 2015
b666820 build: Remove -DBOOST_SPIRIT_THREADSAFE (Wladimir J. van der Laan)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants