[Patch v2 0/4] Staging: vt6655: Simplify statement logic.

* [Patch v2 0/4] Staging: vt6655: Simplify statement logic.
@ 2020-03-31 23:16 Briana Oursler
  2020-03-31 23:16 ` [Patch v2 1/4] Staging: vt6655: Limit return statements Briana Oursler
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Briana Oursler @ 2020-03-31 23:16 UTC (permalink / raw)
  To: julia.lawall, sbrivio, gregkh, forest, outreachy-kernel; +Cc: Briana Oursler

Reduce code duplication and complexity within switch case statement. The
goal is to improve the readability of the statement so that the outcome
of the statement is more clear. Checkpatch identified long lines and
other formatting issues within some nested logic. The last commit in this
patch series addresses checkpatch warnings that were not fixed by other
means within the series. Some long lines were left under the rationale
that removing them rendered the output more confusing.

Briana Oursler (4):
  Staging: vt6655: Limit return statements.
  Staging: vt6655: Move rate determination logic.
  Staging: vt6655: Eliminate nested if else
  Staging: vt6655: Format long lines.

Changes in v2:
	- Patch 1/4:
		- Per Stefano Brivio's recommendation, remove curly
		  braces and unnecessary else clause.
		- Remove unrelated newline.
	- Patch 2/4:
		- Remove unrelated newline.
		- Change to a clearer patch subject and description per
		  Stefano Brivio. 
	- Patch 3/4:
		- Avoid beginning commit description with solution, per
		  Stefano Brivio's advice. 
		- Move declaration of len to the same line as other
		  unsigned ints declared in the same scope, change
		  recommended by Stefano Brivio.
		- Remove unrelated newlines.
	- Patch 4/4:
		- Change patch subject and description to better
		  describe change, per Stefano Brivio. 

 drivers/staging/vt6655/rxtx.c | 131 +++++++++++++++-------------------
 1 file changed, 58 insertions(+), 73 deletions(-)

-- 
2.24.1

^ permalink raw reply	[flat|nested] 9+ messages in thread