squel/steam-manifest
1
1
Fork 0
Code Issues Pull requests Activity Actions

refactor spaghetti code #4

Merged
squel merged 7 commits from squel/refactor into main 2024-11-10 20:51:40 +00:00 AGit
Conversation 0 Commits 7 Files changed 1 +124 -177
squel commented 2024-11-04 17:34:54 +00:00
Owner
Copy link
No description provided. 
- use check_file_data() for file format checks and
offset.
- partly clean STDOUT by either commenting or
spamming `if debug_enabled {}` :P

check_file_data() is previously known as
check_hex_data()
squel changed title from refactor: remove duplicate function (but with difference argument type) to refactor spaghetti code 2024-11-05 01:07:25 +00:00 
seperate the return values of the decode_section()
function, so it can be formatted by the function
that calls it (currently main(), to change)

check the filename hash and return a special error
type. if returned, ignore this specific error:
decode the next section
`<hash>  <path>`, note double space
Instead of checking for bit flips from the cosmic, check for what the
manifest is telling us to check for.
squel merged commit 19077aec6c into main 2024-11-10 20:51:40 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
test/execute

Run CI.

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
test/execute
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
squel/steam-manifest!4
No description provided.