Skip to content

[WIP] Apply pull request #8 from reactos#312

Draft
Copilot wants to merge 5 commits into
udf-devfrom
copilot/udfs-apply-pull-request
Draft

[WIP] Apply pull request #8 from reactos#312
Copilot wants to merge 5 commits into
udf-devfrom
copilot/udfs-apply-pull-request

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 28, 2026

  • udffs.h: add //#define BRUTE comment
  • udf_dbg.h: replace {NOTHING;} / {} no-ops with ((void)0), remove illegal x86 __asm int 3 UDFBreakPoint
  • Include/mem_tools.cpp: replace {NOTHING;} / {} no-ops with ((void)0)
  • udf_info/udf_info.h: rewrite UDFSetBits/UDFClrBits with byte-fill fast path
  • udf_info/alloc.cpp: word-at-a-time bitmap check, bounds-checked bad-space loop, remove redundant debug loops, advance SearchStart

- Replace {NOTHING;}/{} no-op macros with ((void)0) in udf_dbg.h and mem_tools.cpp
- Remove illegal x86 __asm int 3 UDFBreakPoint; use DbgBreakPoint() only
- Add //#define BRUTE comment in udffs.h
- Rewrite UDFSetBits/UDFClrBits with byte-fill fast path (RtlFillMemory/RtlZeroMemory)
- Word-at-a-time bitmap verification in UDFCheckSpaceAllocation_
- Bounds-checked word loop and null-FSBM_Bitmap guard in UDFMarkBadSpaceAsUsed
- Remove redundant UDF_TRACK_ONDISK_ALLOCATION debug assertion loops in UDFMarkSpaceAsXXXNoProtect_
- Advance SearchStart after allocating a fragment in UDFAllocFreeExtent_

Agent-Logs-Url: https://github.com/Zero3K20/reactos/sessions/f5db4087-5531-451c-8404-721a9ea9e9f0

Co-authored-by: Zero3K20 <258969903+Zero3K20@users.noreply.github.com>
Copilot AI mentioned this pull request Apr 30, 2026
16 tasks
Copilot AI added a commit that referenced this pull request May 1, 2026
…chStart advance

Agent-Logs-Url: https://github.com/Zero3K20/reactos/sessions/733f211e-58cf-4005-b7ad-5cae71101dea

Co-authored-by: Zero3K20 <258969903+Zero3K20@users.noreply.github.com>
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.

[UDFS] Apply this PR

3 participants