[Scummvm-git-logs] scummvm master -> bc6b225eed3c43c86a5b216a03b087cba082acd5

sev- noreply at scummvm.org
Mon Jun 5 15:18:31 UTC 2023


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
bc6b225eed COMMON: Include <limits> by default


Commit: bc6b225eed3c43c86a5b216a03b087cba082acd5
    https://github.com/scummvm/scummvm/commit/bc6b225eed3c43c86a5b216a03b087cba082acd5
Author: Eugene Sandulenko (sev at scummvm.org)
Date: 2023-06-05T17:17:19+02:00

Commit Message:
COMMON: Include <limits> by default

We discussed it in https://github.com/scummvm/scummvm/pull/3966, so
std::numeric_limits is OK to use in the ScummVM codebase.

Changed paths:
    common/scummsys.h


diff --git a/common/scummsys.h b/common/scummsys.h
index 09c289f40f8..8fbe5eee816 100644
--- a/common/scummsys.h
+++ b/common/scummsys.h
@@ -137,6 +137,11 @@
 	// easily do that for systems which do have a <new>, since it might clash with
 	// the default definition otherwise!
 	#include <new>
+
+	// After having discussion about use of std::numeric_limits in
+	// https://github.com/scummvm/scummvm/pull/3966 we concluded that it is safe to
+	// use it as minimal STL code with low probability to bring any incompatibilities
+	#include <limits>
 #endif
 
 #ifndef STATIC_ASSERT




More information about the Scummvm-git-logs mailing list