Skip to content

Prepare p14 release #29

Merged
merged 5 commits into from
Jul 20, 2021
Merged

Prepare p14 release #29

merged 5 commits into from
Jul 20, 2021

Conversation

thomas
Copy link
Contributor

@thomas thomas commented Jul 9, 2021

Name it "Radioactive Hybrid Terror Boar from Fukushima" ?

Since string.h is now included, there is no need
to cope with strdup. (Except for keeping a monument
from the 1994 code...)
The routine can be compacted a bit.
 - utilize the scan result from sscanf (as proposed)
 - no need to copy the input to a new buffer, since the
   parts of the path get 'strduped' in parse_entry
Breaking out with exit(0) is way QnD...

This gives the chance to perform further cleanup
when returned into main().
The original xdu used the right button to quit the
application. The double-click variant brings the
feature back.
Copy link
Contributor

@donald donald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. "2 files changed, 147 insertions(+), 166 deletions(-)" . I love PRs which improve functionality AND make the code shorter at the same time.

@wwwutz wwwutz merged commit a14e09c into master Jul 20, 2021
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants