diff --git a/ChangeLog b/ChangeLog index 2316bbbf..a760e626 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2005-12-11 Bart Hakvoort + + * include/GParted_Core.h, + src/GParted_Core.cc: use statvfs instead of this ugly df-wrapping. + 2005-12-08 Bart Hakvoort * include/Partition.h, diff --git a/include/GParted_Core.h b/include/GParted_Core.h index fefee425..387e3629 100644 --- a/include/GParted_Core.h +++ b/include/GParted_Core.h @@ -68,9 +68,9 @@ private: GParted::FILESYSTEM Get_Filesystem( ) ; //temporary function.. asa new checks ripple through in libparted i'll remove it. void set_device_partitions( Device & device ) ; void set_mountpoints( std::vector & partitions, bool first_time = true ) ; + void set_used_sectors( std::vector & partitions ) ; void Insert_Unallocated( const Glib::ustring & device_path, std::vector & partitions, Sector start, Sector end, bool inside_extended ) ; Glib::ustring get_sym_path( const Glib::ustring & real_path ) ; - void Set_Used_Sectors( Partition & partition ); void LP_Set_Used_Sectors( Partition & partition ); Glib::ustring Get_Flags( ) ; int Create_Empty_Partition( Partition & new_partition, bool copy = false ) ; diff --git a/src/GParted_Core.cc b/src/GParted_Core.cc index 1b6005de..256bf3bf 100644 --- a/src/GParted_Core.cc +++ b/src/GParted_Core.cc @@ -1,5 +1,7 @@ #include "../include/GParted_Core.h" +#include + namespace GParted { @@ -121,10 +123,12 @@ void GParted_Core::get_devices( std::vector & devices ) set_device_partitions( temp_device ) ; if ( temp_device .highest_busy ) - { set_mountpoints( temp_device .device_partitions ) ; + + set_used_sectors( temp_device .device_partitions ) ; + + if ( temp_device .highest_busy ) temp_device .readonly = ! ped_disk_commit_to_os( lp_disk ) ; - } } //harddisk without disklabel else @@ -226,21 +230,6 @@ void GParted_Core::set_device_partitions( Device & device ) lp_partition ->type, ped_partition_is_busy( lp_partition ) ); - if ( partition_temp .filesystem != GParted::FS_LINUX_SWAP ) - { - Set_Used_Sectors( partition_temp ) ; - - //the 'Unknown' filesystem warning overrules this one - if ( partition_temp .sectors_used == -1 && partition_temp .error .empty( ) ) - { - partition_temp .error = _("Unable to read the contents of this filesystem!") ; - partition_temp .error += "\n" ; - partition_temp .error += _("Because of this some operations may be unavailable.") ; - partition_temp .error += "\n\n" ; - partition_temp .error += _("Did you install the correct plugin for this filesystem?") ; - } - } - partition_temp .flags = Get_Flags( ) ; if ( partition_temp .busy && partition_temp .partition_number > device .highest_busy ) @@ -325,6 +314,54 @@ void GParted_Core::set_mountpoints( std::vector & partitions, bool fi if ( first_time ) mount_info .clear() ; } + +void GParted_Core::set_used_sectors( std::vector & partitions ) +{ + struct statvfs sfs ; + + temp = _("Unable to read the contents of this filesystem!") ; + temp += "\n" ; + temp += _("Because of this some operations may be unavailable.") ; + temp += "\n\n" ; + temp += _("Did you install the correct plugin for this filesystem?") ; + + for ( unsigned int t = 0 ; t < partitions .size() ; t++ ) + { + if ( partitions[ t ] .filesystem != GParted::FS_LINUX_SWAP && partitions[ t ] .filesystem != GParted::FS_UNKNOWN ) + { + if ( partitions[ t ] .type == GParted::TYPE_PRIMARY || partitions[ t ] .type == GParted::TYPE_LOGICAL ) + { + if ( partitions[ t ] .busy ) + { + if ( statvfs( partitions[ t ] .mountpoint .c_str(), &sfs ) == 0 ) + partitions[ t ] .Set_Unused( sfs .f_bfree * (sfs .f_bsize / 512) ) ; + } + else + { + switch( get_fs( partitions[ t ] .filesystem ) .read ) + { + case GParted::FS::EXTERNAL : + set_proper_filesystem( partitions[ t ] .filesystem ) ; + p_filesystem ->Set_Used_Sectors( partitions[ t ] ) ; + break ; + case GParted::FS::LIBPARTED : + LP_Set_Used_Sectors( partitions[ t ] ) ; + break ; + + default: + break ; + } + } + + if ( partitions[ t ] .sectors_used == -1 ) + partitions[ t ] .error = temp ; + + } + else if ( partitions[ t ] .type == GParted::TYPE_EXTENDED ) + set_used_sectors( partitions[ t ] .logicals ) ; + } + } +} void GParted_Core::Insert_Unallocated( const Glib::ustring & device_path, std::vector & partitions, Sector start, Sector end, bool inside_extended ) { @@ -617,42 +654,6 @@ Glib::ustring GParted_Core::get_sym_path( const Glib::ustring & real_path ) return real_path; } - -void GParted_Core::Set_Used_Sectors( Partition & partition ) -{ - //because 'unknown' is translated we need to call get_fs instead of using partition .filesystem directly - if ( get_fs( partition .filesystem ) .filesystem == GParted::FS_UNKNOWN ) - partition .Set_Unused( -1 ) ; - - else if ( partition .busy ) - { - system( ("df -k --sync " + partition .partition + " | grep " + partition .partition + " > /tmp/.tmp_gparted") .c_str( ) ); - std::ifstream file_input( "/tmp/.tmp_gparted" ); - - //we need the 4th value - file_input >> temp; file_input >> temp; file_input >> temp;file_input >> temp; - if ( ! temp .empty( ) ) - partition .Set_Unused( atol( temp .c_str( ) ) * 2 ) ;// 1024/512 - - file_input .close( ); - system( "rm -f /tmp/.tmp_gparted" ); - } - else - switch( get_fs( partition .filesystem ) .read ) - { - case GParted::FS::EXTERNAL : - set_proper_filesystem( partition .filesystem ) ; - p_filesystem ->Set_Used_Sectors( partition ) ; - break ; - case GParted::FS::LIBPARTED : - LP_Set_Used_Sectors( partition ) ; - break ; - - default: - partition .Set_Unused( -1 ) ; - } -} - void GParted_Core::LP_Set_Used_Sectors( Partition & partition ) { PedFileSystem *fs = NULL;