Skip to content

Conversation

@tritao
Copy link
Contributor

@tritao tritao commented Oct 11, 2024

What does this PR do?

Cleans up warnings from the codebase and some other minor cleanups.

Codebase now compiles without warnings for me.

How does this PR change Premake's behavior?

None.

@tritao tritao force-pushed the cleanups-warnings branch from 8d5bbfe to 5665258 Compare October 11, 2024 13:30
@tritao tritao marked this pull request as ready for review October 11, 2024 13:40
@tritao tritao force-pushed the cleanups-warnings branch from 075190c to 1a80086 Compare October 11, 2024 14:28
@tritao tritao force-pushed the cleanups-warnings branch from 1a80086 to a55adf3 Compare October 11, 2024 14:31
@nickclark2016
Copy link
Member

Can you resolve the conflict and I'll be happy to merge this.

@tritao
Copy link
Contributor Author

tritao commented Oct 25, 2024

Can you resolve the conflict and I'll be happy to merge this.

Done.

@nickclark2016 nickclark2016 merged commit 25fc2e5 into premake:master Oct 26, 2024
15 checks passed
noresources pushed a commit to depinxi/premake-core that referenced this pull request Nov 8, 2024
* Rename `premake_test_file` to `premake_locate_file`.

* Cleanup Windows platform detection.

* Fix warning for Cosmopolitan in `premake_locate_executable`.

* Fix warning in `os_getpass`.

* Update copyright year.

* Preprocessor defines and whitespace cleanup.

* Fix compile warning in `lundunmp.c`.

* Fix compile warning in `mime.c`.

* Fix compile warning in `curl_utils.c`.

* Change `do_path_has_deferred_join` to static.
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.

3 participants