[plasma/kwin] doc: Remove coding-conventions.md

Vlad Zahorodnii <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit ee8ba5d4698b6fb1da8b2b7e29b7588f93c77969 by Vlad Zahorodnii.
Committed on 22/07/2026 at 07:10.
Pushed by vladz into branch 'master'.

Remove coding-conventions.md

It's kind of outdated.

D  +0    -86   doc/coding-conventions.md

https://invent.kde.org/plasma/kwin/-/commit/ee8ba5d4698b6fb1da8b2b7e29b7588f93c77969

diff --git a/doc/coding-conventions.md b/doc/coding-conventions.md
deleted file mode 100644
index 42bb4bfbd4b..00000000000
--- a/doc/coding-conventions.md
+++ /dev/null
@@ -1,86 +0,0 @@
-# Coding Conventions
-
-This document describes some of the recommended coding conventions that should be followed in KWin.
-
-For KWin, it is recommended to follow the KDE Frameworks Coding Style.
-
-
-## `auto` Keyword
-
-Optionally, you can use the `auto` keyword in the following cases. If in doubt, for example if using
-`auto` could make the code less readable, do not use `auto`. Keep in mind that code is read much more
-often than written.
-
-* When it avoids repetition of a type in the same statement.
-
-  ```
-  auto something = new MyCustomType;
-  auto keyEvent = static_cast<QKeyEvent *>(event);
-  auto myList = QStringList({ "FooThing",  "BarThing" });
-  ```
-
-* When assigning iterator types.
-
-  ```
-  auto it = myList.const_iterator();
-  ```
-
-
-## `QRect::right()` and `QRect::bottom()`
-
-For historical reasons, the `QRect::right()` and `QRect::bottom()` functions deviate from the true
-bottom-right corner of the rectangle. Note that this is not the case for the `QRectF` class.
-
-As a general rule, avoid using `QRect::right()` and `QRect::bottom()` as well methods that operate
-on them. There are exceptions, though.
-
-Exception 1: you can use `QRect::moveRight()` and `QRect::moveBottom()` to snap a `QRect` to
-another `QRect` as long as the corresponding borders match, for example
-
-```
-// Ok
-rect.moveRight(anotherRect.right());
-rect.moveBottom(anotherRect.bottom());
-rect.moveBottomRight(anotherRect.bottomRight());
-
-// Bad
-rect.moveRight(anotherRect.left() - 1); // must be rect.moveLeft(anotherRect.left() - rect.width());
-rect.moveBottom(anotherRect.top() - 1); // must be rect.moveTop(anotherRect.top() - rect.height());
-rect.moveBottomRight(anotherRect.topLeft() - QPoint(1, 1));
-```
-
-Exception 2: you can use `QRect::setRight()` and `QRect::setBottom()` to clip a `QRect` by another
-`QRect` as long as the corresponding borders match, for example
-
-```
-// Ok
-rect.setRight(anotherRect.right());
-rect.setBottom(anotherRect.bottom());
-rect.setBottomRight(anotherRect.bottomRight());
-
-// Bad
-rect.setRight(anotherRect.left());
-rect.setBottom(anotherRect.top());
-rect.setBottomRight(anotherRect.topLeft());
-```
-
-Exception 3: you can use `QRect::right()` and `QRect::bottom()` in conditional statements as long
-as the compared borders are the same, for example
-
-```
-// Ok
-if (rect.right() > anotherRect.right()) {
-    return;
-}
-if (rect.bottom() > anotherRect.bottom()) {
-    return;
-}
-
-// Bad
-if (rect.right() > anotherRect.left()) {
-    return;
-}
-if (rect.bottom() > anotherRect.top()) {
-    return;
-}
-```
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.