fix: add missing newlines in logs (#77)

FrogTheFrog · web-flow · commit 3d584f096c25 · 2024-07-30T14:19:45.000Z
diff --git a/src/windows/settings_manager_apply.cpp b/src/windows/settings_manager_apply.cpp
@@ -244,7 +244,7 @@ namespace display_device {
       const auto original_primary_device { cached_primary_device.empty() ? current_primary_device : cached_primary_device };
       const auto &new_primary_device { device_to_configure };
 
-      if (!try_change(new_primary_device, "Changing primary display to: ", "Failed to apply new configuration, because a new primary device could not be set!")) {
+      if (!try_change(new_primary_device, "Changing primary display to:\n", "Failed to apply new configuration, because a new primary device could not be set!")) {
         // Error already logged
         return false;
       }
@@ -255,7 +255,7 @@ namespace display_device {
     }
 
     if (might_need_to_restore) {
-      if (!try_change(cached_primary_device, "Changing primary display back to: ", "Failed to restore original primary device!")) {
+      if (!try_change(cached_primary_device, "Changing primary display back to:\n", "Failed to restore original primary device!")) {
         // Error already logged
         return false;
       }
@@ -307,7 +307,7 @@ namespace display_device {
       const auto new_display_modes { win_utils::computeNewDisplayModes(config.m_resolution,
         config.m_refresh_rate, configuring_primary_devices, device_to_configure, additional_devices_to_configure, original_display_modes) };
 
-      if (!try_change(new_display_modes, "Changing display modes to: ", "Failed to apply new configuration, because new display modes could not be set!")) {
+      if (!try_change(new_display_modes, "Changing display modes to:\n", "Failed to apply new configuration, because new display modes could not be set!")) {
         // Error already logged
         return false;
       }
@@ -318,7 +318,7 @@ namespace display_device {
     }
 
     if (might_need_to_restore) {
-      if (!try_change(cached_display_modes, "Changing display modes back to: ", "Failed to restore original display modes!")) {
+      if (!try_change(cached_display_modes, "Changing display modes back to:\n", "Failed to restore original display modes!")) {
         // Error already logged
         return false;
       }
@@ -364,7 +364,7 @@ namespace display_device {
       const auto original_hdr_states { cached_hdr_states.empty() ? current_hdr_states : cached_hdr_states };
       const auto new_hdr_states { win_utils::computeNewHdrStates(config.m_hdr_state, configuring_primary_devices, device_to_configure, additional_devices_to_configure, original_hdr_states) };
 
-      if (!try_change(new_hdr_states, "Changing HDR states to: ", "Failed to apply new configuration, because new HDR states could not be set!")) {
+      if (!try_change(new_hdr_states, "Changing HDR states to:\n", "Failed to apply new configuration, because new HDR states could not be set!")) {
         // Error already logged
         return false;
       }
@@ -375,7 +375,7 @@ namespace display_device {
     }
 
     if (might_need_to_restore) {
-      if (!try_change(cached_hdr_states, "Changing HDR states back to: ", "Failed to restore original HDR states!")) {
+      if (!try_change(cached_hdr_states, "Changing HDR states back to:\n", "Failed to restore original HDR states!")) {
         // Error already logged
         return false;
       }

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ namespace display_device {`
`244`	`244`	`const auto original_primary_device { cached_primary_device.empty() ? current_primary_device : cached_primary_device };`
`245`	`245`	`const auto &new_primary_device { device_to_configure };`
`246`	`246`
`247`		`- if (!try_change(new_primary_device, "Changing primary display to: ", "Failed to apply new configuration, because a new primary device could not be set!")) {`
	`247`	`+ if (!try_change(new_primary_device, "Changing primary display to:\n", "Failed to apply new configuration, because a new primary device could not be set!")) {`
`248`	`248`	`// Error already logged`
`249`	`249`	`return false;`
`250`	`250`	`}`
`@@ -255,7 +255,7 @@ namespace display_device {`
`255`	`255`	`}`
`256`	`256`
`257`	`257`	`if (might_need_to_restore) {`
`258`		`- if (!try_change(cached_primary_device, "Changing primary display back to: ", "Failed to restore original primary device!")) {`
	`258`	`+ if (!try_change(cached_primary_device, "Changing primary display back to:\n", "Failed to restore original primary device!")) {`
`259`	`259`	`// Error already logged`
`260`	`260`	`return false;`
`261`	`261`	`}`
`@@ -307,7 +307,7 @@ namespace display_device {`
`307`	`307`	`const auto new_display_modes { win_utils::computeNewDisplayModes(config.m_resolution,`
`308`	`308`	`config.m_refresh_rate, configuring_primary_devices, device_to_configure, additional_devices_to_configure, original_display_modes) };`
`309`	`309`
`310`		`- if (!try_change(new_display_modes, "Changing display modes to: ", "Failed to apply new configuration, because new display modes could not be set!")) {`
	`310`	`+ if (!try_change(new_display_modes, "Changing display modes to:\n", "Failed to apply new configuration, because new display modes could not be set!")) {`
`311`	`311`	`// Error already logged`
`312`	`312`	`return false;`
`313`	`313`	`}`
`@@ -318,7 +318,7 @@ namespace display_device {`
`318`	`318`	`}`
`319`	`319`
`320`	`320`	`if (might_need_to_restore) {`
`321`		`- if (!try_change(cached_display_modes, "Changing display modes back to: ", "Failed to restore original display modes!")) {`
	`321`	`+ if (!try_change(cached_display_modes, "Changing display modes back to:\n", "Failed to restore original display modes!")) {`
`322`	`322`	`// Error already logged`
`323`	`323`	`return false;`
`324`	`324`	`}`
`@@ -364,7 +364,7 @@ namespace display_device {`
`364`	`364`	`const auto original_hdr_states { cached_hdr_states.empty() ? current_hdr_states : cached_hdr_states };`
`365`	`365`	`const auto new_hdr_states { win_utils::computeNewHdrStates(config.m_hdr_state, configuring_primary_devices, device_to_configure, additional_devices_to_configure, original_hdr_states) };`
`366`	`366`
`367`		`- if (!try_change(new_hdr_states, "Changing HDR states to: ", "Failed to apply new configuration, because new HDR states could not be set!")) {`
	`367`	`+ if (!try_change(new_hdr_states, "Changing HDR states to:\n", "Failed to apply new configuration, because new HDR states could not be set!")) {`
`368`	`368`	`// Error already logged`
`369`	`369`	`return false;`
`370`	`370`	`}`
`@@ -375,7 +375,7 @@ namespace display_device {`
`375`	`375`	`}`
`376`	`376`
`377`	`377`	`if (might_need_to_restore) {`
`378`		`- if (!try_change(cached_hdr_states, "Changing HDR states back to: ", "Failed to restore original HDR states!")) {`
	`378`	`+ if (!try_change(cached_hdr_states, "Changing HDR states back to:\n", "Failed to restore original HDR states!")) {`
`379`	`379`	`// Error already logged`
`380`	`380`	`return false;`
`381`	`381`	`}`