Commit graph

5 commits

Author SHA1 Message Date
Vadim Peretokin
b23d6f788e
Infrastructure: fix else-after-return in codebase (#9096)
<!-- Keep the title short & concise so anyone non-technical can
understand it,
     the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Remove unnecessary else/else-if after return, break, continue, and throw
statements - and in places where fixing them is more trouble than its
worth, added NOLINT.
#### Motivation for adding to Mudlet

https://clang.llvm.org/extra/clang-tidy/checks/readability/else-after-return.html,
so it doesn't pop up in PR reviews.
#### Other info (issues closed, discussion etc)

---------

Co-authored-by: Stephen Lyons <slysven@virginmedia.com>
Co-authored-by: Vadim Peretokin <vadi2@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
2026-07-18 18:39:57 +02:00
Vadim Peretokin
b6738dd8c2
infrastructure: Apply clang-format to all CPP files (#8804)
#### Brief overview of PR changes/additions
Ran clang-format on all 134 CPP files in src/ using the project's
.clang-format config

#### Motivation for adding to Mudlet
Ensures consistent code formatting across the codebase.

#### Other info (issues closed, discussion etc)
None

**Test case:** Build the project and verify it compiles successfully.
2026-01-19 18:10:44 +01:00
Vadim Peretokin
8cb58cf0ca
Infrastructure: remove MSVC leak detection (#8378)
<!-- Keep the title short & concise so anyone non-technical can
understand it,
     the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Remove support for leak detection using MSVC - this functionality hasn't
been in use in a decade, and we're switching to to using
[LeakSanitizer](https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizer)
instead.
#### Motivation for adding to Mudlet
Cleaner code.
#### Other info (issues closed, discussion etc)

---------

Co-authored-by: Claude <noreply@anthropic.com>
2025-10-21 15:11:09 +01:00
Stephen Lyons
92a483cad4
Infrastructure: clean up missing line-feeds at start of methods (#7803)
#### Brief overview of PR changes/additions
This PR adds a missing Line-Feed to those methods that are missing them
after the closing `)` or the `) const` of the arguments to that method.

#### Motivation for adding to Mudlet
We intend for the opening `{` to be in the first column of the next line
but some PRs in the past have added methods (functions) where this
hasn't been done and this is not something that the `clang-format` that
we bundle can fix-up for us automatically.

#### Other info (issues closed, discussion etc)

Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
2025-04-17 02:22:26 +01:00
Mike Conley
f56291b6c2
Fix: Replace QMediaPlaylist for move to Qt 6 (#7210)
<!-- Keep the title short & concise so anyone non-technical can
understand it,
     the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions

To move to Qt 6, Mudlet must eliminate a dependency on QMediaPlaylist. A
revised implementation is included.

#### Motivation for adding to Mudlet

The update to Qt6 removed QMediaPlaylist support

#### Other info (issues closed, discussion etc)

Author: Tamarindo@StickMUD

API call for testing:
```lua
lua playMusicFile({name = "167124__patricia-mcmillen__rugby-club-in-spain.mp3", volume = 100, fadein = 20000, fadeout = 53000, loops = 2, continue = true})
```
2024-04-28 08:24:12 +02:00