Remove leading newlines from user-oriented output

They don't make much sense and make it harder to be parsed.

Even though user-oriented output is not meant to be parsed, it is very common
to have user-oriented output be more or less machine-friendly. It actually is
the case of the Parameter Framework's remote commands, except for these leading
newlines.

Signed-off-by: David Wagner <[email protected]>

Author: dawagner

parameter/ConfigurableDomain.cpp

@@ -686,8 +686,6 @@ bool CConfigurableDomain::deleteConfiguration(const string &strName, string &str
 
 void CConfigurableDomain::listAssociatedToElements(string &strResult) const
 {
-    strResult = "\n";
-
     ConfigurableElementListIterator it;
 
     // Browse all configurable elements

parameter/ConfigurableDomains.cpp

@@ -317,8 +317,6 @@ bool CConfigurableDomains::split(const string &domainName, CConfigurableElement
 
 void CConfigurableDomains::listAssociatedElements(string &strResult) const
 {
-    strResult = "\n";
-
     std::set<const CConfigurableElement *> configurableElementSet;
 
     // Get all owned configurable elements
@@ -341,8 +339,6 @@ void CConfigurableDomains::listAssociatedElements(string &strResult) const
 
 void CConfigurableDomains::listConflictingElements(string &strResult) const
 {
-    strResult = "\n";
-
     std::set<const CConfigurableElement *> configurableElementSet;
 
     // Get all owned configurable elements
@@ -369,8 +365,6 @@ void CConfigurableDomains::listConflictingElements(string &strResult) const
 
 void CConfigurableDomains::listDomains(string &strResult) const
 {
-    strResult = "\n";
-
     // List domains
     size_t uiNbConfigurableDomains = getNbChildren();
 

parameter/ConfigurableElement.cpp

@@ -468,8 +468,6 @@ void CConfigurableElement::listBelongingDomains(std::string &strResult, bool bVe
 // Elements with no domains
 void CConfigurableElement::listRogueElements(std::string &strResult) const
 {
-    strResult = "\n";
-
     // Get rogue element aggregate list (no associated domain)
     std::list<const CConfigurableElement *> rogueElementList;
 
@@ -568,11 +566,6 @@ void CConfigurableElement::listDomains(
     const std::list<const CConfigurableDomain *> &configurableDomainList, std::string &strResult,
     bool bVertical) const
 {
-    if (bVertical && configurableDomainList.empty()) {
-
-        strResult = "\n";
-    }
-
     // Fill list
     ConfigurableDomainListConstIterator it;
     bool bFirst = true;

parameter/Element.cpp

@@ -122,7 +122,6 @@ string CElement::dumpContent(utility::ErrorContext &errorContext, const size_t d
 // Element properties
 void CElement::showProperties(string &strResult) const
 {
-    strResult = "\n";
     strResult += "Kind: " + getKind() + "\n";
     showDescriptionProperty(strResult);
 }
@@ -330,8 +329,6 @@ bool CElement::removeChild(CElement *pChild)
 
 void CElement::listChildren(string &strChildList) const
 {
-    strChildList = "\n";
-
     // Get list of children names
     for (CElement *pChild : _childArray) {
 

parameter/ParameterMgr.cpp

@@ -1370,8 +1370,6 @@ CParameterMgr::CCommandHandler::CommandStatus CParameterMgr::listElementsCommand
         return CCommandHandler::EFailed;
     }
 
-    strResult = string("\n");
-
     if (!pLocatedElement) {
 
         // List from root folder
@@ -1399,8 +1397,6 @@ CParameterMgr::CCommandHandler::CommandStatus CParameterMgr::listParametersComma
         return CCommandHandler::EFailed;
     }
 
-    strResult = string("\n");
-
     if (!pLocatedElement) {
 
         // List from root folder

remote-processor/RemoteCommandHandlerTemplate.h

@@ -194,8 +194,6 @@ class TRemoteCommandHandlerTemplate : public IRemoteCommandHandler
     {
         initMaxCommandUsageLength();
 
-        strResult = "\n";
-
         // Show usages
         for (const auto *pRemoteCommandParserItem : _remoteCommandParserVector) {