Added two comments, to remind myself to check something.

This commit is contained in:
John Wiegley 2009-02-06 23:37:25 -04:00
parent cdcd52dbaf
commit aea84dd9c5
2 changed files with 3 additions and 0 deletions

View file

@ -205,6 +205,8 @@ namespace {
report_t& report(find_scope<report_t>(args)); report_t& report(find_scope<report_t>(args));
item_t& item(find_scope<item_t>(args)); item_t& item(find_scope<item_t>(args));
// jww (2009-02-06): Should we be calling reported_date here?
date_t when; date_t when;
if (report.HANDLED(effective)) { if (report.HANDLED(effective)) {
if (optional<date_t> date = item.effective_date()) if (optional<date_t> date = item.effective_date())

View file

@ -38,6 +38,7 @@ namespace ledger {
date_t xact_t::date() const date_t xact_t::date() const
{ {
// jww (2009-02-06): Why isn't reported_date called here?
if (! _date) { if (! _date) {
assert(entry); assert(entry);
return entry->date(); return entry->date();