Skip to content
Snippets Groups Projects
Commit d2404851 authored by Adriaan de Groot's avatar Adriaan de Groot
Browse files

[partition] Reduce warnings (shadow, ;, types)

parent 1a4a2652
Branches
Tags
No related merge requests found
...@@ -64,10 +64,10 @@ namespace Choices ...@@ -64,10 +64,10 @@ namespace Choices
quint64 requiredSpaceB; // estimated required space for root partition quint64 requiredSpaceB; // estimated required space for root partition
SwapChoice swap; SwapChoice swap;
AutoPartitionOptions( const QString& fs, const QString& luks, const QString& efi, qint64 r, SwapChoice s ) AutoPartitionOptions( const QString& fs, const QString& luks, const QString& efi, qint64 requiredBytes, SwapChoice s )
: ReplacePartitionOptions( fs, luks ) : ReplacePartitionOptions( fs, luks )
, efiPartitionMountPoint( efi ) , efiPartitionMountPoint( efi )
, requiredSpaceB( r > 0 ? r : 0 ) , requiredSpaceB( requiredBytes > 0 ? static_cast<quint64>( requiredBytes ) : 0 )
, swap( s ) , swap( s )
{ {
} }
......
...@@ -60,8 +60,8 @@ PartitionLayout::PartitionLayout( PartitionLayout::PartitionEntry entry ) ...@@ -60,8 +60,8 @@ PartitionLayout::PartitionLayout( PartitionLayout::PartitionEntry entry )
} }
PartitionLayout::PartitionLayout( const PartitionLayout& layout ) PartitionLayout::PartitionLayout( const PartitionLayout& layout )
: m_partLayout( layout.m_partLayout ) : m_defaultFsType( layout.m_defaultFsType )
, m_defaultFsType( layout.m_defaultFsType ) , m_partLayout( layout.m_partLayout )
{ {
} }
......
...@@ -51,7 +51,7 @@ public: ...@@ -51,7 +51,7 @@ public:
PartUtils::SizeUnit partMaxSizeUnit = PartUtils::SizeUnit::Percent; PartUtils::SizeUnit partMaxSizeUnit = PartUtils::SizeUnit::Percent;
/// @brief All-zeroes PartitionEntry /// @brief All-zeroes PartitionEntry
PartitionEntry() {}; PartitionEntry() {}
/// @brief Parse @p size, @p min and @p max to their respective member variables /// @brief Parse @p size, @p min and @p max to their respective member variables
PartitionEntry( const QString& size, const QString& min, const QString& max ); PartitionEntry( const QString& size, const QString& min, const QString& max );
}; };
......
...@@ -65,7 +65,7 @@ PartitionModel::init( Device* device , const OsproberEntryList& osproberEntries ...@@ -65,7 +65,7 @@ PartitionModel::init( Device* device , const OsproberEntryList& osproberEntries
} }
int int
PartitionModel::columnCount( const QModelIndex& parent ) const PartitionModel::columnCount( const QModelIndex& ) const
{ {
return ColumnCount; return ColumnCount;
} }
...@@ -247,7 +247,7 @@ PartitionModel::data( const QModelIndex& index, int role ) const ...@@ -247,7 +247,7 @@ PartitionModel::data( const QModelIndex& index, int role ) const
} }
QVariant QVariant
PartitionModel::headerData( int section, Qt::Orientation orientation, int role ) const PartitionModel::headerData( int section, Qt::Orientation, int role ) const
{ {
if ( role != Qt::DisplayRole ) if ( role != Qt::DisplayRole )
return QVariant(); return QVariant();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment