| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
|
|
|
|
The memory for this format is now statically allocated as well. I
was under the impression its size would depend on wincols, but this is
the format we're talking about, the string does not have to hold the
actual line contents. I must have been sleeping again...
Oh well, this is a slight performance improvement, although it doesn't
seem the be the cause of the browing slowness when running under
valgrind. (Obviously running ncdu with valgrind is supposed to be
slower, but the current performance is rather bad...)
|
|
Instead you now get the usual 'no items to display' message when hiding
hidden files in the root directory.
|
|
This changes the behaviour back to it was before the dirlist
abstraction, which is the behaviour I prefer.
|
|
Rather than storing a pointer to another memory allocation in the
struct. This saves some memory and improves performance by significantly
decreasing the number of calls to [c|m]alloc() and free().
|
|
Here is the new multi-page listing functionality I promised in
5db9c2aea11052451c7e11bf8eef73393e4a072e.
It may look very easy, but getting this to work right wasn't,
unfortunately.
|
|
Also changed other occurences of the same situation to use the same
checking method (>= rather than a +1) for consistency.
Fixes bug #2981704.
|
|
Rather than the ugly hack in delete.c.
|
|
This significantly improves the performance when browsing large
directories. It is somewhat costly on the memory usage, though. :-(
|
|
I was overseeing a stupid mistake in
a7b7841ac084b21afd85c85c896fe89c441d4062
Obviously, you need to check against wincols and not winrows...
|
|
|
|
Fixes a tiny, insignificant memory leak.
|
|
Fixes bug #2991787
|
|
The parent directory reference is the same for all directories, so we'll
have to make sure to reset its FF_BSEL flag when opening another
directory.
|
|
This optimizes a few actions (though not all), and makes the code easier
to understand and expand.
The behaviour of the browser has changed a bit with regards to
multi-page listings. Personally I don't like this change much, so I'd
probably fix that later on.
|
|
|
|
The displayed directory sizes are now fully correct, although in its
current state it's not all that intuitive because:
directory size != sum(size of all files and subdirectories)
This should probably be fixed later on by splitting the sizes into a
shared and non-shared part.
Also, the sizes displayed after a recalculation or deletion are
incorrect, I'll fix this later on.
|
|
|
|
|
|
The directory sizes are now incorrect as hard links will be counted
twice again (as if there wasn't any detection in the first place), but
this will get fixed by adding a shared size field.
This method of keeping track of hard links is a lot faster and allows
adding an interface which lists the found links.
|
|
|
|
|
|
|
|
This (in combination with linking to ncursesw) fixes the display of
non-ASCII characters.
|
|
Or force the use of a library with --with-ncurses or --with-ncursesw.
Implementation based on/stolen from the configure.ac in ncmpc.
|
|
These ideas may not be very easy to implement, and may not be worth
the performance penalty they might introduce. But that's something that
still needs to be investigated.
|
|
Fixes bug #2831412 (debian #539553)
|
|
...instead of displaying an empty and unusable browser.
|
|
When interrupinting the calculation process by pressing 'q' while
it's looping through a directory, or when a directory could be openend
but not chdir()'ed into, closedir() wasn't called.
|
|
This fixes a tiny memory leak as well, as the return value of opendir()
wasn't passed to a closedir() after use.
|
|
|
|
Fixes bug #2789781.
|
|
|
|
There shouldn't be a need for such a state when there's a central
place where the program execution keeps returning to.
|
|
|
|
|
|
Setting FF_BSEL after calling browse_init() causes two items to be
selected, as browse_init() makes sure something will be selected,
while calc_process() assumes nothing is, because the previously
selected item had just been deleted.
|
|
Hard link detection is now done in a separate pass on the in-memory tree,
and duplicates can be 'removed' and 're-added' on the fly. When making any
changes in the tree, all hard links are re-added before the operation and
removed again afterwards.
While this guarantees that all hard link information is correct, it does
have a few drawbacks. I can currently think of two:
1. It's not the most efficient way to do it, and may be quite slow on
large trees. Will have to do some benchmarks later to see whether
it is anything to be concerned about.
2. The first encountered item is considered as 'counted' and all items
encountered after that are considered as 'duplicate'. Because the
order in which we traverse the tree doesn't always have to be the
same, the items that will be considered as 'duplicate' can vary with
each deletion or re-calculation. This might cause confusion for
people who aren't aware of how hard links work.
|
|
I happen to make these kinds of mistakes a lot, for some strange
reason. Maybe I should just get more sleep...
|
|
|
|
|
|
|
|
Reproducable with the following steps:
- Make sure the first item in the list is a directory starting with a dot
- Make sure the next item is a normal directory
- Select first item
- Press 'h' to hide it, next item will properly be selected
- Open selected directory (right arrow), and see how the first (hidden)
directory was opened.
It's amazing how I've even found this bug, considering that really is the
only way to reproduce it...
|
|
|
|
|
|
|
|
It's a floating point division, so won't really cause any real problems
on most systems. Still, a percentage of 'nan' isn't really useful.
|
|
Which would happen if there's only one file is in a directory
|
|
Removed some obsolete macros and added a few more headers and function
checks.
|
