Ticket #188 (closed enhancement: fixed)
(mandriva) show free space
| Reported by: | metux | Owned by: | styx |
|---|---|---|---|
| Priority: | major | Milestone: | 4.7 |
| Component: | mc-core | Version: | 4.6.2 |
| Keywords: | commited-master | Cc: | |
| Blocked By: | Blocking: | ||
| Branch state: | Votes for changeset: |
Description (last modified by metux) (diff)
Attachments
Change History
comment:2 Changed 16 years ago by andrew_b
I've modified this patch (see attachment). Differences:
- info format is changed from (as an example) "5M (50%) of 10M" to " 5M/10M (50%) ". So we don't need to translate it.
- show_free_space() function is moved from main.c to screen.c. Now this function is called in show_dir() function only. Related global variables are moved into show_free_space() and made static.
But there is a problem as in Mandriva/RH patch as in mine one. The init_my_statfs() function is called many times, therefore we have the memory leak.
comment:3 Changed 16 years ago by metux
- Owner set to metux
- Keywords vote-metux added
- Status changed from new to accepted
- Description modified (diff)
comment:4 Changed 16 years ago by andrew_b
- Version changed from 4.6.1 to 4.6.2
I fixed memory leak in init_my_statfs() function. Patch is attached. Please review.
Changed 16 years ago by andrew_b
- Attachment mountlist-free_mount_list.patch added
Fix of memory leak in init_my_statsf()
comment:6 Changed 16 years ago by styx
I think original patch needs to be reworked. Some const's for options count like in option.c is done. Why new option has index 0? This one makes useless changes in source code. It should be with the last index...
comment:7 Changed 16 years ago by styx
my vote for mountlist-free_mount_list.patch but not for branch 188_show_free_space
Please, don't merge this into master.
comment:10 Changed 16 years ago by styx
- Status changed from accepted to assigned
- Owner changed from metux to styx
comment:13 Changed 16 years ago by metux
- Owner changed from styx to metux
- Description modified (diff)
rebased to master:
changeset:186cb259c08be7a955298b57b060dd1a1aff6617
comment:16 Changed 16 years ago by winnie
Why changed you the owner from styx to you? Please don't do that in future. If someone has accepted a bug he works on it.
comment:18 Changed 16 years ago by styx
- Status changed from accepted to assigned
- Owner changed from metux to styx
comment:19 Changed 16 years ago by styx
- Keywords approved commited-master added
- Status changed from assigned to testing
- Resolution set to fixed
