[PR #8995] Manually initialize the warnings vector to prevent a crash on launch #27365

Open
opened 2026-01-31 09:21:33 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/8995

State: closed
Merged: Yes


Summary of the Pull Request

Oops, winrt IVectors need to be manually initialized, when default-constructed std::vectors didn't. Simple oversight.

PR Checklist

Validation Steps Performed

Ran the terminal with


    "schemes" :
    [ {} ]

First the crash repro'd, now it doesn't.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/8995 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Oops, winrt `IVector`s need to be manually initialized, when default-constructed `std::vector`s didn't. Simple oversight. ## PR Checklist * [x] Closes #8986 * [x] I work here * [x] A test would be great but ain't nobody got time for that. * [n/a] Requires documentation to be updated ## Validation Steps Performed Ran the terminal with ```json "schemes" : [ {} ] ``` First the crash repro'd, now it doesn't.
claunia added the pull-request label 2026-01-31 09:21:33 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#27365