Used boost::shared_ptr throughout for memory management #1

Open
wants to merge 35 commits into
from

Conversation

Projects
None yet
1 participant
Owner

cce commented Apr 7, 2013

This PR brings in changes from the long-running shared_ptr feature branch, merged with the latest from brownie/master. It replaces all the new & delete with reference-counted boost::shared_ptr & boost::make_shared.

cce and others added some commits Dec 18, 2010

@cce cce converting to use boost::shared_ptr (as Ptr), ran query-replace-regex…
… '\(\(const \)?[A-Za-z:]+\)\( \)?\*\( \)?\([^/]\)' with 'Ptr<\1> \5'
948b95f
@cce cce use new_ptr instead of new (query-replace-regexp 'new \([A-Za-z:_]+\)…
…' -> 'new_ptr<\1>' in all files)
46145d6
@cce cce run in ZKP dir: sed -i.bak s/boost::shared_ptr/Ptr/g *.cpp *.h *.hpp 7972845
@cce cce clarify (mini)NTL and license for ZZ.c fdcf2ec
@cce cce Merge branch 'master' into shared_ptr 82f2d96
@cce cce [shared_ptr branch] got rid of deletes, unnecessary copy constructors…
…. seems to compile..
3fa150e
@cce cce running tests (not all pass), rebuilt param files 8108abd
@cce cce trying to fix fbeeaaf
@cce cce imitate the way copying new groups worked with old BankParameters 1edc573
@cce cce use loadString instead of loadGZString for client tests df9db21
@cce cce params from master 80a4991
@cce cce get rid of buymX.xml 261b44c
@cce cce use std::map to fix serialization tests for now c68957c
@cce cce Merge remote branch 'origin/shared_ptr' into broken-shared_ptr
Conflicts:
	src/GroupRSA.h
	src/Makefile
	src/Test.cpp
	src/bank.80.params
	src/public.80.arbiter
	src/public.regular.80.arbiter
	src/secret.80.arbiter
	src/secret.regular.80.arbiter
	src/tool.80.bank
	src/tool.80.user
	src/wallet.80
a0d3922
@cce cce Revert "add operator== testers to some classes (for debugging)"
This reverts commit bcb2114.
7ab320f
@cce cce fix interleaved/repeated withdraw tests with shared_ptr 0c3d2da
@cce cce regenerate params with std::map and shared_ptr 245890e
@cce cce shared_ptr seems to work! 96a9ee6
@cce cce fixed tests 13 and 14
Conflicts:

	src/bank.80.params
	src/tool.80.bank
	src/wallet.80
9be3034
@cce Sarah Meiklejohn getting rid of some residual debugging stuff 4857032
@cce cce remove seller destructor eeff6a4
@cce cce new unordered_map serialization 1efbc9c
@cce cce check for CACHE_POWERS environment variable f7caf46
@cce cce Merge branch 'shared_ptr' of github.com:brownie/cashlib into shared_ptr 62274b0
@cce cce fix serialize ZZ=0 problem (did mpz_sizeinbase change?) c9a212b
@cce cce print out difference in len/l 3163f13
@cce cce deserialize messages from GZ-compressed strings by default 72b60ee
@cce cce Revert "print out difference in len/l"
This reverts commit 3163f13.
02ceba5
@cce cce fixed test 10
Conflicts:

	src/bank.80.params
	src/public.80.arbiter
	src/public.regular.80.arbiter
	src/secret.80.arbiter
	src/secret.regular.80.arbiter
	src/tool.80.bank
	src/tool.80.user
	src/wallet.80
6618484
@cce cce make brownie timers thread-safe (use boost::thread_specific_ptr) 7ad4c0b
@cce cce allow option to Interpreter::check() to enable/disable multiexponenti…
…ation power caching
777e759
@cce cce skip power-caching for user-to-bank proofs by default 56bc7cf
@cce cce make InterpreterCache thread-safe (use TLS from boost::thread_specifi…
…c_ptr)
e469ced
@cce cce Merge remote-tracking branch 'origin/master' into shared_ptr-merge
resolved conflicts with git mergetool & DiffMerge

Conflicts:
	src/BuyMessage.h
	src/Buyer.cpp
	src/Debug.h
	src/FEInitiator.cpp
	src/FEResponder.cpp
	src/GroupRSA.h
	src/Seller.cpp
	src/Test.cpp
	src/ZKP/Interpreter.cpp
	src/ZKP/examples/cl-obtain-temp.txt
	src/bank.80.params
	src/tool.80.bank
	src/wallet.80
b7667a7
@cce cce fix build after merge (remove copy constructors again) 4440e91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment