Anastasia Klimchuk has uploaded this change for review.

View Change

tests: Trim excessive new lines from layout test output

This patch is making layout test output more compact, spanning
less vertical lines while logging the same information.

No changes made to the test functionality.

BUG=b:193584590
TEST=ninja test

Change-Id: I36a39505f8db508672cde5a848684a3a22f5ccc4
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
---
M tests/layout.c
1 file changed, 18 insertions(+), 18 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/39/56439/1

diff --git a/tests/layout.c b/tests/layout.c
index e3e9539..03a61c7 100644
--- a/tests/layout.c
+++ b/tests/layout.c
@@ -23,26 +23,26 @@
 {
 	(void) state; /* unused */
 
-	printf("Creating layout...\n");
+	printf("Creating layout...");
 	struct flashrom_layout *layout;
 	assert_int_equal(0, flashrom_layout_new(&layout));
 	printf("done\n");
 
-	printf("Adding and including first region...\n");
+	printf("Adding and including first region...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00021000, 0x00031000, "first region"));
 	assert_int_equal(0, flashrom_layout_include_region(layout, "first region"));
-	printf("done\n");
+	printf("done");
 
-	printf("Adding and including second (non-overlapping) region...\n");
+	printf(", second (non-overlapping) region...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00031001, 0x0023efc0, "second region"));
 	assert_int_equal(0, flashrom_layout_include_region(layout, "second region"));
 	printf("done\n");
 
-	printf("Asserting included regions do not overlap...\n");
+	printf("Asserting included regions do not overlap...");
 	assert_int_equal(0, included_regions_overlap(layout));
 	printf("done\n");
 
-	printf("Releasing layout...\n");
+	printf("Releasing layout...");
 	flashrom_layout_release(layout);
 	printf("done\n");
 }
@@ -51,26 +51,26 @@
 {
 	(void) state; /* unused */
 
-	printf("Creating layout...\n");
+	printf("Creating layout...");
 	struct flashrom_layout *layout;
 	assert_int_equal(0, flashrom_layout_new(&layout));
 	printf("done\n");
 
-	printf("Adding and including first region...\n");
+	printf("Adding and including first region...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00021000, 0x00031000, "first region"));
 	assert_int_equal(0, flashrom_layout_include_region(layout, "first region"));
-	printf("done\n");
+	printf("done");
 
-	printf("Adding and including second (overlapping) region...\n");
+	printf(", second (overlapping) region...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00027100, 0x0023efc0, "second region"));
 	assert_int_equal(0, flashrom_layout_include_region(layout, "second region"));
 	printf("done\n");
 
-	printf("Asserting included regions overlap...\n");
+	printf("Asserting included regions overlap...");
 	assert_int_equal(1, included_regions_overlap(layout));
 	printf("done\n");
 
-	printf("Releasing layout...\n");
+	printf("Releasing layout...");
 	flashrom_layout_release(layout);
 	printf("done\n");
 }
@@ -79,25 +79,25 @@
 {
 	(void) state; /* unused */
 
-	printf("Creating layout...\n");
+	printf("Creating layout...");
 	struct flashrom_layout *layout;
 	assert_int_equal(0, flashrom_layout_new(&layout));
 	printf("done\n");
 
-	printf("Adding and including first region...\n");
+	printf("Adding and including first region...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00021000, 0x00031000, "first region"));
 	assert_int_equal(0, flashrom_layout_include_region(layout, "first region"));
-	printf("done\n");
+	printf("done");
 
-	printf("Adding second (overlapping) region, not included...\n");
+	printf(", second (overlapping) region, not included...");
 	assert_int_equal(0, flashrom_layout_add_region(layout, 0x00027100, 0x0023efc0, "second region"));
 	printf("done\n");
 
-	printf("Asserting included regions do not overlap...\n");
+	printf("Asserting included regions do not overlap...");
 	assert_int_equal(0, included_regions_overlap(layout));
 	printf("done\n");
 
-	printf("Releasing layout...\n");
+	printf("Releasing layout...");
 	flashrom_layout_release(layout);
 	printf("done\n");
 }

To view, visit change 56439. To unsubscribe, or for help writing mail filters, visit settings.