napcok/calamares
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[libcalamares] Improve naming in implementation

Browse Source 
- The code in loadYaml was refactored out of the module-descriptor
   loading code, but the variable names in the implementation were
   not changed and still strangely specific to the prior task.
This commit is contained in:
Adriaan de Groot 2019-01-25 11:59:17 -05:00
parent b3a7545217
commit 5efbf51ed3
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/libcalamares/utils/YamlUtils.cpp
View File

@ -173,15 +173,15 @@ loadYaml(const QString& filename, bool* ok)
if ( ok )
*ok = false;
QFile descriptorFile( filename );
QVariant moduleDescriptor;
if ( descriptorFile.exists() && descriptorFile.open( QFile::ReadOnly | QFile::Text ) )
QFile yamlFile( filename );
QVariant yamlContents;
if ( yamlFile.exists() && yamlFile.open( QFile::ReadOnly | QFile::Text ) )
{
QByteArray ba = descriptorFile.readAll();
QByteArray ba = yamlFile.readAll();
try
{
YAML::Node doc = YAML::Load( ba.constData() );
moduleDescriptor = CalamaresUtils::yamlToVariant( doc );
yamlContents = CalamaresUtils::yamlToVariant( doc );
}
catch ( YAML::Exception& e )
{
@ -191,13 +191,13 @@ loadYaml(const QString& filename, bool* ok)
}
if ( moduleDescriptor.isValid() &&
!moduleDescriptor.isNull() &&
moduleDescriptor.type() == QVariant::Map )
if ( yamlContents.isValid() &&
!yamlContents.isNull() &&
yamlContents.type() == QVariant::Map )
{
if ( ok )
*ok = true;
return moduleDescriptor.toMap();
return yamlContents.toMap();
}
return QVariantMap();