Norm Evangelista

Changed memcpy/memset for non-POD objects [PCL-3368]

14

Per [C++ Coding Standards item 96](https://www.oreilly.com/library/view/c-coding-standards/0321113586/ch97.html): > `memcpy` and `memcmp` violate the type system. Using `memcpy` to copy objects is like making money using a photocopier. Using `memcmp` to compare...

[custom] Remove empty destructors and constructors

2

This issue is to split PR #4249 into smaller pieces. Per @SergioRAgostinho > With destructors I use the following policy: > > if your class defines virtual methods which it...

Modernized memset via uniform init or modern constructs

2

As a follow-up to #4299, modernized more `memset` usages by either employing uniform initialization or modern constructs

Switched to generic endian swap

13

Thanks a lot for making a contribution to SFML! 🙂 Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs...

Adds the [bugprone-macro-parentheses](https://clang.llvm.org/extra/clang-tidy/checks/bugprone/macro-parentheses.html) check to `clang-tidy`: >Finds macros that can have unexpected behavior due to missing parentheses. There are macros that use `std::stringstream` to simulate variadic functions; these should probably...

Part A of clang-tidy bugprone-implicit-widening-of-multiplication-result

3

Part A of support for the `clang-tidy` check [`bugprone-implicit-widening-of-multiplication-result`](https://clang.llvm.org/extra/clang-tidy/checks/bugprone/implicit-widening-of-multiplication-result.html). As with PR #5665, this check will be implemented over several PRs to keep them to a reasonable size.