obsproject / obs-studio

OBS Studio - Free and open source software for live streaming and screen recording
https://obsproject.com
GNU General Public License v2.0
60.39k stars 7.99k forks source link

various: Remove HAVE_OBSCONFIG_H ifdefs #11472

Open gxalpha opened 2 weeks ago

gxalpha commented 2 weeks ago

Description

Removes HAVE_OBSCONFIG_H ifdef-checks and marks the definition for removal in 32.0 (as removing it could be considered a breaking change, in case there are any third-party plugins checking for it).

Motivation and Context

With the removal of all legacy code paths, obsconfig.h always exists and the HAVE_OBSCONFIG_H compile definition always gets set. As such, it's no longer necessary to check for it.

How Has This Been Tested?

Tested with CI that all builds still compile. Tested with git grep that the definition line in cmake is now the only instance remaining of HAVE_OBSCONFIG_H.

Types of changes

Checklist:

norihiro commented 2 weeks ago

I'm sorry that there is no module name various which is written in the commit message.