Skip to content

Remove UTF-8 BOM from NativeEngine.cpp#1623

Merged
bkaradzic-microsoft merged 2 commits intochange-threading-modelfrom
copilot/sub-pr-1620
Mar 10, 2026
Merged

Remove UTF-8 BOM from NativeEngine.cpp#1623
bkaradzic-microsoft merged 2 commits intochange-threading-modelfrom
copilot/sub-pr-1620

Conversation

Copy link

Copilot AI commented Mar 9, 2026

Plugins/NativeEngine/Source/NativeEngine.cpp had a UTF-8 BOM (EF BB BF) prepended, which can break toolchains and linters that expect plain ASCII/UTF-8 without BOM.

Changes

  • Stripped the three-byte UTF-8 BOM from the start of NativeEngine.cpp so the file begins directly with #include "NativeEngine.h"

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: bkaradzic-microsoft <260535795+bkaradzic-microsoft@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on bgfx initialization move to JS thread Remove UTF-8 BOM from NativeEngine.cpp Mar 9, 2026
@bkaradzic-microsoft bkaradzic-microsoft marked this pull request as ready for review March 10, 2026 14:05
@bkaradzic-microsoft bkaradzic-microsoft merged commit 9b6d591 into change-threading-model Mar 10, 2026
3 checks passed
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.

2 participants