Skip to content

events: remove unneeded safety check code

Rodrigo Muino Tomonari requested to merge github/fork/captainsafia/remove-unneccessary-event-code into master
Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

events

Description of change

In the process of creating #9865, it was discovered that the code checking whether or not events was defined was unnecessary because the only situation in which events would be undefined is if it is monkeypatched by an external entity. This should be removed in order to discourage this. If the test added in #9865 is merged, it will need to removed on merge of this commit.

Before merging this change we might want to check and see if any significant usage of monkeypatching events in the ecosystem.

Merge request reports