Commit Graph

3546 Commits

Author SHA1 Message Date
ZXShady
72207a63ef Use inherited constructors instead of constructor delegation
sf::Exception can use the inheriting constructors functionality instead of a constructor delegating to the parent constructor.

I removed Exception.cpp as it is not needed anymore.
2024-10-05 10:22:49 -06:00
ZXShady
9169e11195 Remove unneeded class prefixes 2024-10-05 09:41:45 -06:00
ZXShady
8fd67a0d99 Remove wrong comment
There is no explicit scope made specifically for an RAII variable, so I removed the comment
2024-10-05 13:37:06 +01:00
ZXShady
cfc43f50a7 Use sf::VertexArray.clear instead of .resize(0) 2024-10-04 10:43:35 -06:00
Lukas Dürrenberger
ecb945b341 Use explicit constructors when implicit construction isn't wanted 2024-10-03 22:43:57 -06:00
vittorioromeo
816855d13f Remove sf::Vector2<T>::Unit[XY] 2024-10-03 20:51:09 -06:00
Chris Thrasher
a9298df466 Upgrade all macOS 12 CI jobs to macOS 13 2024-10-03 19:13:55 -06:00
Chris Thrasher
c67e835e81 Remove Android CI unit testing infrastructure
Initially we had 3 of 4 Android CI jobs running the tests. Various
problems caused us to disable testing in 2 of those 3 jobs. Now
we're dealing with the last Android CI job failing to run the tests.
Because this is blocking CI and because the value of running these
tests it not especially high, I am removing this testing
infrastructure. We can reconsider this decision in the future if a
more reliable solution is found.

This is a reversion of most of 2386653.
2024-10-03 14:34:07 -06:00
ZXShady
de9286c11f Use explicit conversion operator instead of explicit constructor
I also added a type trait test to check whether the conversion operator is marked explicit.

this change has the advantage of being  clearer imo as it is a conversion operator for conersions not a constructor.

and also it allows users to take the address of convesion operator which I don't know if someone might find that useful.
2024-10-02 00:15:54 +02:00
Chris Thrasher
4e77e3ee95 Revert to LLVM 18 until we can upgrade to macOS 14 images 2024-10-01 14:51:41 -06:00
ZXShady
c8dbd65ff5 use if constexpr instead of a switch statement
The condition is a constant expression, it makes more sense here to use `if constexpr`
2024-10-01 00:00:48 -06:00
vittorioromeo
37de949d87 Actually, don't define 2024-09-30 23:15:18 -06:00
ZXShady
71594d402e delegate to operator equals in operator not equals 2024-09-30 09:03:35 -06:00
Chris Thrasher
ea08388e7a Remove unnecessary ternaries
It's simpler to let the boolean get promoted to an integer
2024-09-29 23:34:27 -06:00
Lukas Dürrenberger
053ef0b483 Prefer try_emplace over emplace for maps 2024-09-27 10:10:23 -06:00
vittorioromeo
eb4a2dbe0f Drain errors in [E]GLCheck via loop 2024-09-27 10:09:07 -06:00
vittorioromeo
7cec342b2b DRY shader creation via helper lambda 2024-09-26 16:33:05 -06:00
vittorioromeo
a9c56da99e Fix island example freeze when submitting work too quickly 2024-09-26 18:04:32 +02:00
Chris Thrasher
d2ca038e2a Actually let's not 2024-09-25 20:37:43 -06:00
Chris Thrasher
9d1e5d9acf Prevent reading from an uninitialized character buffer
If `GLEXT_glGetInfoLog` fails then `log` is left unchanged which
can lead to a reading from an uninitialized buffer on the following
line. Initializing the buffer ensures this never happens. While I
was touching this code I elected to switch to `std::array` as well
since SFML has made previous efforts to stop using C-style arrays
and I wanted to continue that effort.
2024-09-25 19:34:16 -06:00
ZXShady
0d33ddeccf Change glCheck macro to be usable as an expression 2024-09-25 09:47:36 -06:00
Chris Thrasher
c8cf84511d Fix clang-tidy-19 errors 2024-09-24 18:42:31 -06:00
Vittorio Romeo
1f11e91bed Avoid unnecessary assignment in Text::findCharacterPos 2024-09-22 14:20:25 +02:00
kimci86
267bbe35b8 Simplify island example using vectors and extracting repetitive code 2024-09-17 22:38:51 +02:00
Lorenzooone
6bcc3414fc Use lazy loading for keyboard scancodes on macos 2024-09-16 16:35:12 -04:00
Vittorio Romeo
20b048fb08 Replace swap with move where appropriate 2024-09-16 13:29:52 -04:00
Lukas Dürrenberger
6591c59504 Add window create functions without style specifier
This brings the create() functions inline with the constructor overloads
for WindowBase, Window, and RenderWindow
2024-09-16 12:05:59 -04:00
Vittorio Romeo
e2ea1d8a7c Simplify and extend [E]GLCheck implementation 2024-09-16 12:02:54 -04:00
Lukas Dürrenberger
2dd72be036 Fix Windows ARM64 support with regards to Mesa 3D
- Only fail the Mesa 3D arch check, if it's been enabled
- Add a cross-compilation CI job for Windows ARM64
- Fix ARM64 architecture detection for the Ninja generator
2024-09-14 13:44:34 +02:00
Lukas Dürrenberger
8ff95f7a5e Add the missing ChannelMap to the example code 2024-09-14 09:11:16 +02:00
Lukas Dürrenberger
2116a3ba85 Extend the migration guide regarding the sf::Text constructor change 2024-09-12 17:54:34 -06:00
Chris Thrasher
eeb43e4747 Use sf::Vector3f to simplify Vulkan example 2024-09-12 15:18:44 -06:00
Chris Thrasher
44b5237148 Fix out of date documentation code 2024-09-12 14:37:07 -06:00
Chris Thrasher
7eab4606e0 Use standard CMake variables for Doxygen paths 2024-09-12 12:21:09 -06:00
Chris Thrasher
acc87dfd32 Simplify how Doxygen CMake variables are set 2024-09-12 12:21:09 -06:00
Vittorio Romeo
9c3fcd72d0 Use gender-neutral language in comments 2024-09-12 17:37:10 +02:00
Lukas Dürrenberger
016e201d17 Use backticks for SFML types in doc comments 2024-09-11 16:02:01 -06:00
Chris Thrasher
5e56f01612 Use macOS to build documentation 2024-09-11 16:02:01 -06:00
Chris Thrasher
ab109cd3cd Use ON and OFF for CMake booleans
CMake supports a number of strings for truthy and falsey values.
ON/OFF and TRUE/FALSE are the most popular but 1/0 is also supported.
This is mostly a style choice but I'm inclined to believe that ON/OFF
is the most popular option and I'm generally in favor of style
choices that better align with the community at large.
2024-09-11 14:36:13 -06:00
Chris Thrasher
157d4d9163 Stop using deprecated DOXYGEN_EXECUTABLE variable
This was deprecated in CMake 3.9.

See https://cmake.org/cmake/help/v3.22/module/FindDoxygen.html#variable:DOXYGEN_EXECUTABLE
2024-09-11 13:48:12 -06:00
Chris Thrasher
7fdce1235e Link to migration guide from changelog 2024-09-10 12:03:01 -06:00
Chris Thrasher
2b4a72ee06 Fix more outdated references to Mac OS X 2024-09-09 23:42:26 -06:00
Lukas Dürrenberger
9f52cfec39 Change spelling of anti-aliasing 2024-09-09 18:37:00 -06:00
Lukas Dürrenberger
265a0cb03f Rename Vector2 & Vector3 functions to better fit the naming convention 2024-09-08 12:09:32 -06:00
Chris Thrasher
61d78105e3 Assert against C-style string arguments being null 2024-09-05 22:02:55 -06:00
Chris Thrasher
74dfd76b25 Pass sf::Vector2<T> by value in test code 2024-09-01 17:51:54 +01:00
Chris Thrasher
73e5abe583 Fix typo in migration guide 2024-09-01 08:59:58 -06:00
Chris Thrasher
b55327b5b9 Remove unused header 2024-08-31 16:46:47 -06:00
Chris Thrasher
a7f7026804 Fix Doxygen warning 2024-08-31 12:02:59 -06:00
Chris Thrasher
8ed98e337b Treat Doxygen warnings as errors 2024-08-31 12:02:59 -06:00