summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Löfman <ext-robert.lofman@nokia.com>2010-12-01 11:51:22 +0200
committerRobert Löfman <ext-robert.lofman@nokia.com>2010-12-01 11:51:22 +0200
commit248ec24e0e776588278053c33bef18be52c489c2 (patch)
treeedea2b391273ffa2b407ae129e5992990d8d3dcf
parentbd39d64e2e0c08f323187fe9946c437b5ea8af35 (diff)
Implemented suggestions from reviews.
-rw-r--r--libresourceqt/src/resource-set.cpp6
-rw-r--r--resourceqt-client/client.cpp56
-rw-r--r--resourceqt-client/client.h2
3 files changed, 27 insertions, 37 deletions
diff --git a/libresourceqt/src/resource-set.cpp b/libresourceqt/src/resource-set.cpp
index 8f26a13..086401c 100644
--- a/libresourceqt/src/resource-set.cpp
+++ b/libresourceqt/src/resource-set.cpp
@@ -230,14 +230,12 @@ Resource * ResourceSet::resource(ResourceType type) const
bool ResourceSet::initAndConnect()
{
- if ( !initialized )
- {
+ if ( !initialized ) {
qDebug("ResourceSet::%s().... initializing...", __FUNCTION__);
return initialize();
}
- if ( !resourceEngine->isConnectedToManager() )
- {
+ if ( !resourceEngine->isConnectedToManager() ) {
qDebug("ResourceSet::%s().... connecting...", __FUNCTION__);
return resourceEngine->connectToManager();
}
diff --git a/resourceqt-client/client.cpp b/resourceqt-client/client.cpp
index 23d5acb..ee76bf8 100644
--- a/resourceqt-client/client.cpp
+++ b/resourceqt-client/client.cpp
@@ -138,7 +138,7 @@ bool Client::initialize(const CommandLineParser &parser)
start_timer();
resourceSet->initAndConnect();
- output << "connecting...accepting input" << endl;
+ output << "accepting input" << endl;
showPrompt();
return true;
}
@@ -150,72 +150,64 @@ void Client::doExit()
}
-void Client::modifyResources(QString resString)
+void Client::modifyResources(const QString &resString)
{
//resString example: [mand_resources:opt_resources] res1,res2,res3:res1,res3
- if ( resString.isEmpty() || resString.isNull())
- {
+ if ( resString.isEmpty() || resString.isNull()){
qDebug("Client::modifyResources(): no resources in string.");
return;
}
QStringList newAllAndOpt = resString.split(":");
if (newAllAndOpt.size()==1)
- qDebug("There are only manditory resources.");
+ qDebug("There are only mandatory resources.");
//Every optional res. is also in allSet
QSet<ResourcePolicy::ResourceType> newAllSet;
QSet<ResourcePolicy::ResourceType> newOptSet;
- if ( !CommandLineParser::parseResourceList(newAllAndOpt.at(0), newAllSet) )
+ if ( !CommandLineParser::parseResourceList(newAllAndOpt.at(0), newAllSet) ) {
+ qDebug("Client::modifyResources(): could not parse all resources.");
return;
+ }
- if (newAllAndOpt.size()>1)
- {
- if ( !CommandLineParser::parseResourceList(newAllAndOpt.at(1), newOptSet) )
+ if (newAllAndOpt.size()>1) {
+ if ( !CommandLineParser::parseResourceList(newAllAndOpt.at(1), newOptSet) ){
+ qDebug("Client::modifyResources(): could not parse optional resources.");
return;
+ }
+ bool optNotInAll = false;
//If the user forgot to add resource to all when specifying optional -> add to all.
- foreach(ResourceType newOptRes, newOptSet)
- {
- if ( !newAllSet.contains(newOptRes) )
- {
- qDebug("Client::modifyResources(): optional resources should be added to all as well.");
+ foreach(ResourceType newOptRes, newOptSet){
+ if ( !newAllSet.contains(newOptRes) ){
+ optNotInAll = true;
newAllSet.insert(newOptRes);
}
}
+ if (optNotInAll)
+ qDebug("Client::modifyResources(): optional resources should be added to all as well.");
}
- QSet<ResourceType>::iterator newResIt = newAllSet.begin();
- QList<Resource*> resList = resourceSet->resources();
-
//Check if new resources are in current resource set.
- while (newResIt != newAllSet.end())
- {
- ResourceType newRes = *newResIt;
-
- if ( resourceSet->contains(newRes) )
- {
+ foreach ( ResourceType newRes, newAllSet) {
+ if ( resourceSet->contains(newRes) ){
if ( resourceSet->resource(newRes)->isOptional() && !newOptSet.contains(newRes) ) {
- //New manditory is in set, but is not optional in the new set.
+ //New mandatory is in set, but is not optional in the new set.
resourceSet->resource(newRes)->setOptional(false);
}
- else if ( !resourceSet->resource(newRes)->isOptional() && newOptSet.contains(newRes) )
- {
- //New manditory is in set, but is optional.
+ else if ( !resourceSet->resource(newRes)->isOptional() && newOptSet.contains(newRes) ){
+ //New mandatory is in set, but is optional.
resourceSet->resource(newRes)->setOptional(true);
}
}
- else
- { //Add new resource.
+ else { //Add new resource.
resourceSet->addResource(newRes);
if ( newOptSet.contains(newRes) )
resourceSet->resource(newRes)->setOptional(true);
-
}
- ++newResIt;
}
-
+ QList<Resource*> resList = resourceSet->resources();
//Check if there are current resources not in the new set (i.e. removed).
foreach(Resource* resource, resList)
if ( !newAllSet.contains(resource->type()) )
diff --git a/resourceqt-client/client.h b/resourceqt-client/client.h
index b2aca41..e592fb9 100644
--- a/resourceqt-client/client.h
+++ b/resourceqt-client/client.h
@@ -80,7 +80,7 @@ private:
void showPrompt();
void showResources(const QList<ResourcePolicy::ResourceType> &resList);
void showResources(const QList<ResourcePolicy::Resource*> &resList);
- void modifyResources(QString resString);
+ void modifyResources(const QString &resString);
};
QTextStream & operator<< (QTextStream &output,