[PR #753] [MERGED] Try first LBA + 5 as well when determining readcd in case of first LB… #1144

Open
opened 2026-01-29 15:42:26 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/aaru-dps/Aaru/pull/753
Author: @FakeShemp
Created: 10/14/2022
Status: Merged
Merged: 10/20/2024
Merged by: @claunia

Base: develHead: rebecca/first-lba-broken-workaround


📝 Commits (2)

  • fca596f Try first LBA + 5 as well when determining readcd in case of first LBA being bad
  • b845252 Merge branch 'devel' into rebecca/first-lba-broken-workaround

📊 Changes

1 file changed (+41 additions, -60 deletions)

View changed files

📝 Aaru.Core/Devices/Dumping/CompactDisc/Dump.cs (+41 -60)

📄 Description

…A being bad

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • New filesystem, test images in [url]
  • New media image, test images in [url]
  • New partition scheme, test images in [url]
  • Breaking change (fix or feature that would cause existing functionality to change)
  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/aaru-dps/Aaru/pull/753 **Author:** [@FakeShemp](https://github.com/FakeShemp) **Created:** 10/14/2022 **Status:** ✅ Merged **Merged:** 10/20/2024 **Merged by:** [@claunia](https://github.com/claunia) **Base:** `devel` ← **Head:** `rebecca/first-lba-broken-workaround` --- ### 📝 Commits (2) - [`fca596f`](https://github.com/aaru-dps/Aaru/commit/fca596f1243872cf6fee1adc4cc6cca05f61b5c0) Try first LBA + 5 as well when determining readcd in case of first LBA being bad - [`b845252`](https://github.com/aaru-dps/Aaru/commit/b845252e58c8cb4f4ba2fb4253efead91a33dd97) Merge branch 'devel' into rebecca/first-lba-broken-workaround ### 📊 Changes **1 file changed** (+41 additions, -60 deletions) <details> <summary>View changed files</summary> 📝 `Aaru.Core/Devices/Dumping/CompactDisc/Dump.cs` (+41 -60) </details> ### 📄 Description …A being bad ## Types of changes <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [X] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] New filesystem, test images in [url] - [ ] New media image, test images in [url] - [ ] New partition scheme, test images in [url] - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [X] I have read the **CONTRIBUTING** document. - [X] My code follows the code style of this project. - [ ] My change requires a change to the documentation. - [ ] I have updated the documentation accordingly. - [ ] I have added tests to cover my changes. - [ ] All new and existing tests passed. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:42:27 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aaru-dps/Aaru-aaru-dps#1144