binsider icon indicating copy to clipboard operation
binsider copied to clipboard

Added human readable option for Tab::StaticAnalysis

Open dacianpascu06 opened this issue 1 year ago β€’ 1 comments

Description of change

Added KeyCommand HumanReadable that toggles the format for FileSiz and MemSiz in Tab::StaticAnalysis => Program Headers. Keybind is 's'

Closes Issue #47

How has this been tested? (if applicable)

Verified the program to correctly toggle human readable on and off only on tab StaticAnalysis.

dacianpascu06 avatar Oct 09 '24 07:10 dacianpascu06

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 3.31%. Comparing base (4bc6b7a) to head (6e1ea2f). Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
src/elf/header.rs 0.00% 13 Missing :warning:
src/tui/state.rs 0.00% 5 Missing :warning:
src/tui/command.rs 0.00% 1 Missing :warning:
Additional details and impacted files
@@           Coverage Diff            @@
##            main     #77      +/-   ##
========================================
- Coverage   3.33%   3.31%   -0.01%     
========================================
  Files         19      19              
  Lines       2407    2417      +10     
========================================
  Hits          80      80              
- Misses      2327    2337      +10     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 10 '24 08:10 codecov[bot]

@orhun (don't think force push sends notif)

dacianpascu06 avatar Oct 15 '24 09:10 dacianpascu06