Skip to content

rename SectorReadMode to SectorReadFormat#50

Merged
Bloomca merged 1 commit into
mainfrom
rename-sector-read-struct
Jul 12, 2026
Merged

rename SectorReadMode to SectorReadFormat#50
Bloomca merged 1 commit into
mainfrom
rename-sector-read-struct

Conversation

@Bloomca

@Bloomca Bloomca commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Description

Rename the SectorReadMode into a more specific:

pub enum SectorReadFormat {
    /// Audio: 2352 bytes/sector, raw PCM.
    /// CDB byte 1 = 0x00 (any type), byte 9 = 0x10 (user data).
    Audio,
    /// Mode 1 cooked data: 2048 bytes/sector, containing user data only.
    /// CDB byte 1 = 0x08 (Mode 1), byte 9 = 0x10 (user data).
    Mode1Cooked,
    /// Complete Mode 1 sector: 2352 bytes with sync, header, user data, EDC, and ECC.
    /// CDB byte 1 = 0x08 (Mode 1), byte 9 = 0xF8 (all main-channel fields).
    Mode1Raw,
}

@Bloomca
Bloomca merged commit 5b4d8af into main Jul 12, 2026
@Bloomca
Bloomca deleted the rename-sector-read-struct branch July 12, 2026 06:36
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.

1 participant