staging: rtl8192u: cleanup of ternary and if else statements
Get series with b4
b4 am [email protected]Versions newest first
- v2 Latest2 patches 9 repliesIn review
Changes in v2 from the cover letter
* Patch 1: "staging: rtl8192u: replace ternary statement with if and
* assignment"
* replaced max macro with an if statement followed by an assignment
* Patch 2: "staging: rtl8192u: use min_t/max_t macros instead of if
* else"
* changed the type argument in min_t and max_t from u8 to u32
---
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | staging: rtl8192u: replace ternary statement with if and assignmentdrivers/ | +3 | 8 replies | |
| 2/2 | staging: rtl8192u: use min_t/max_t macros instead of if elsedrivers/ | +5 | 0 replies |
Reviews trailers per patch
None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.
| Patch | Dan Carpenter |
|---|---|
| Series | R-b |
| 1/2 | |
| 2/2 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 0/2] staging: rtl8192u: cleanup of ternary and if else statements
These patches replace ternary and if else statements with more readable statements. Found with minmax coccinelle script.
---
v1 -> v2:
* Patch 1: "staging: rtl8192u: replace ternary statement with if and
* assignment"
* replaced max macro with an if statement followed by an assignment
* Patch 2: "staging: rtl8192u: use min_t/max_t macros instead of if
* else"
* changed the type argument in min_t and max_t from u8 to u32
---
Rebecca Mckeever (2):
staging: rtl8192u: replace ternary statement with if and assignment
staging: rtl8192u: use min_t/max_t macros instead of if else
drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c | 4 +++- drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 13 +++++-------- 2 files changed, 8 insertions(+), 9 deletions(-)
Signature · 1 line
2.32.0