Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

BDecisionProvider: Make YesNoDecisionNeeded() non-abstract.

Browse Source 
- A subset of jobs that require a BContext don't in fact make use of the
decision provider. As such, make the default implementation usable for
those cases so one doesn't need to always create a dummy derived class.
This commit is contained in:
Rene Gollent 2013-09-27 21:06:34 -04:00
parent a7a7aaaf3d
commit 672795a37e
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
headers/os/package/Context.h
View File

@ -25,7 +25,7 @@ struct BDecisionProvider {
virtual bool YesNoDecisionNeeded(const BString& description, virtual bool YesNoDecisionNeeded(const BString& description,
const BString& question, const BString& question,
const BString& yes, const BString& no, const BString& yes, const BString& no,
const BString& defaultChoice) = 0; const BString& defaultChoice);
// virtual bool ActionsAcceptanceDecisionNeeded( // virtual bool ActionsAcceptanceDecisionNeeded(
// const BString& description, // const BString& description,
// const BString& question) = 0; // const BString& question) = 0;

9
src/kits/package/Context.cpp
View File

@ -29,6 +29,15 @@ BDecisionProvider::~BDecisionProvider()
} }
bool
BDecisionProvider::YesNoDecisionNeeded(const BString& description,
const BString& question, const BString& yes, const BString& no,
const BString& defaultChoice)
{
return defaultChoice == yes;
}
BContext::BContext(BDecisionProvider& decisionProvider, BContext::BContext(BDecisionProvider& decisionProvider,
BJobStateListener& jobStateListener) BJobStateListener& jobStateListener)
: :