Skip to content

Comments

Rewrite ExFAT implementation#37

Open
Horofic wants to merge 1 commit intomainfrom
refactor-exfat
Open

Rewrite ExFAT implementation#37
Horofic wants to merge 1 commit intomainfrom
refactor-exfat

Conversation

@Horofic
Copy link
Member

@Horofic Horofic commented Dec 19, 2025

This PR rewrites/refactors the current ExFAT implementation. As the previous one was not functional.

closes #23

def __init__(self, fh: bytes | memoryview | bytearray | BinaryIO, /): ...

# Technically `c_fat` is an instance of `_c_fat`, but then we can't use it in type hints
c_fat: TypeAlias = _c_fat
Copy link
Member

Choose a reason for hiding this comment

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

Missing constants.

Copy link
Member

Choose a reason for hiding this comment

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

Most of these changes are already in main, what's up with that?

def __init__(self, fh: bytes | memoryview | bytearray | BinaryIO, /): ...

# Technically `c_exfat` is an instance of `_c_exfat`, but then we can't use it in type hints
c_exfat: TypeAlias = _c_exfat
Copy link
Member

Choose a reason for hiding this comment

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

Missing constants.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rewrite ExFAT implementation

2 participants