sfos/ledger
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixed another memory strangeness (too much destruction)

Browse source
This commit is contained in:
John Wiegley 2004-08-26 02:01:29 -04:00
parent 69bd31b4d0
commit 5f30c790db
3 changed files with 48 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
value.cc
View file

@ -26,6 +26,7 @@ value_t& value_t::operator=(const value_t& value)
return *this;
destroy();
switch (value.type) {
case BOOLEAN:
*((bool *) data) = *((bool *) value.data);

69
value.h
View file

@ -37,7 +37,7 @@ class value_t
type = INTEGER;
}
value_t(const value_t& value) {
value_t(const value_t& value) : type(INTEGER) {
DEBUG_PRINT("ledger.memory.ctors", "ctor value_t");
*this = value;
}
@ -51,20 +51,17 @@ class value_t
*((unsigned int *) data) = value;
type = INTEGER;
}
value_t(const amount_t& value) {
value_t(const amount_t& value) : type(INTEGER) {
DEBUG_PRINT("ledger.memory.ctors", "ctor value_t");
new((amount_t *)data) amount_t(value);
type = AMOUNT;
*this = value;
}
value_t(const balance_t& value) {
value_t(const balance_t& value) : type(INTEGER) {
DEBUG_PRINT("ledger.memory.ctors", "ctor value_t");
new((balance_t *)data) balance_t(value);
type = BALANCE;
*this = value;
}
value_t(const balance_pair_t& value) {
value_t(const balance_pair_t& value) : type(INTEGER) {
DEBUG_PRINT("ledger.memory.ctors", "ctor value_t");
new((balance_pair_t *)data) balance_pair_t(value);
type = BALANCE_PAIR;
*this = value;
}
~value_t() {
@ -76,33 +73,55 @@ class value_t
value_t& operator=(const value_t& value);
value_t& operator=(const bool value) {
destroy();
*((bool *) data) = value;
type = BOOLEAN;
if ((bool *) data != &value) {
destroy();
*((bool *) data) = value;
type = BOOLEAN;
}
return *this;
}
value_t& operator=(const unsigned int value) {
destroy();
*((unsigned int *) data) = value;
type = INTEGER;
if ((unsigned int *) data != &value) {
destroy();
*((unsigned int *) data) = value;
type = INTEGER;
}
return *this;
}
value_t& operator=(const amount_t& value) {
destroy();
new((amount_t *)data) amount_t(value);
type = AMOUNT;
if ((amount_t *) data != &value) {
if (! value) {
return *this = 0U;
} else {
destroy();
new((amount_t *)data) amount_t(value);
type = AMOUNT;
}
}
return *this;
}
value_t& operator=(const balance_t& value) {
destroy();
new((balance_t *)data) balance_t(value);
type = BALANCE;
if ((balance_t *) data != &value) {
if (value.amounts.size() == 1) {
return *this = (*value.amounts.begin()).second;
} else {
destroy();
new((balance_t *)data) balance_t(value);
type = BALANCE;
}
}
return *this;
}
value_t& operator=(const balance_pair_t& value) {
destroy();
new((balance_pair_t *)data) balance_pair_t(value);
type = BALANCE_PAIR;
if ((balance_pair_t *) data != &value) {
if (! value.cost) {
return *this = value.quantity;
} else {
destroy();
new((balance_pair_t *)data) balance_pair_t(value);
type = BALANCE_PAIR;
}
}
return *this;
}

6
walk.h
View file

@ -40,6 +40,9 @@ struct item_handler {
template <typename T>
class compare_items {
value_t left_result;
value_t right_result;
const value_expr_t * sort_order;
public:
@ -52,9 +55,6 @@ class compare_items {
assert(left);
assert(right);
value_t left_result;
value_t right_result;
sort_order->compute(left_result, details_t(left));
sort_order->compute(right_result, details_t(right));