RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 22:57:44 +00:00
Code Issues Activity Actions

Kernel: Make enumerate_disk_partitions function not const

Browse source 
The enumerate_disk_partitions function doesn't need to be const. Remove
the constness and use the newly added `add_partition` function.
This commit is contained in:
Pankaj Raghav 2022-01-03 16:28:38 +05:30 • committed by Brian Gianforcaro
parent 2c810332b6
commit 9ae2285675
2 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Kernel/Storage/StorageManagement.cpp
View file

@ -110,10 +110,9 @@ UNMAP_AFTER_INIT OwnPtr<PartitionTable> StorageManagement::try_to_initialize_par
return {};
}
UNMAP_AFTER_INIT void StorageManagement::enumerate_disk_partitions() const
UNMAP_AFTER_INIT void StorageManagement::enumerate_disk_partitions()
{
VERIFY(!m_storage_devices.is_empty());
NonnullRefPtrVector<DiskPartition> partitions;
size_t device_index = 0;
for (auto& device : m_storage_devices) {
auto partition_table = try_to_initialize_partition_table(device);
@ -124,9 +123,8 @@ UNMAP_AFTER_INIT void StorageManagement::enumerate_disk_partitions() const
if (!partition_metadata.has_value())
continue;
// FIXME: Try to not hardcode a maximum of 16 partitions per drive!
auto disk_partition = DiskPartition::create(const_cast<StorageDevice&>(device), (partition_index + (16 * device_index)), partition_metadata.value());
MUST(partitions.try_append(disk_partition));
MUST(const_cast<StorageDevice&>(device).m_partitions.try_append(disk_partition));
auto disk_partition = DiskPartition::create(device, (partition_index + (16 * device_index)), partition_metadata.value());
device.add_partition(disk_partition);
}
device_index++;
}