Skip to content

Format fix - #6025

Closed
JurajSadel wants to merge 1 commit into
mainfrom
fmt
Closed

Format fix#6025
JurajSadel wants to merge 1 commit into
mainfrom
fmt

Conversation

@JurajSadel

Copy link
Copy Markdown
Contributor

The fmt check is currently failing:

[2026-08-02T12:18:59Z INFO  esp_devtool] Formatting package: esp-alloc
Diff in /home/runner/work/esp-hal/esp-hal/esp-alloc/src/lib.rs:475:
         }
 
         cfg_select! {
-            feature = "internal-heap-stats" => {
-                HeapStats {
-                    region_stats,
-                    size: free + used,
-                    current_usage: used,
-                    max_usage: self.internal_heap_stats.max_usage,
-                    total_allocated: self.internal_heap_stats.total_allocated,
-                    total_freed: self.internal_heap_stats.total_freed,
-                }
-            }
-            _ => {
-                HeapStats {
-                    region_stats,
-                    size: free + used,
-                    current_usage: used,
-                }
-            }
+            feature = "internal-heap-stats" => HeapStats {
+                region_stats,
+                size: free + used,
+                current_usage: used,
+                max_usage: self.internal_heap_stats.max_usage,
+                total_allocated: self.internal_heap_stats.total_allocated,
+                total_freed: self.internal_heap_stats.total_freed,
+            },
+            _ => HeapStats {
+                region_stats,
+                size: free + used,
+                current_usage: used,
+            },
         }
     }
 
Diff in /home/runner/work/esp-hal/esp-hal/esp-alloc/src/lib.rs:475:
         }
 
         cfg_select! {
-            feature = "internal-heap-stats" => {
-                HeapStats {
-                    region_stats,
-                    size: free + used,
-                    current_usage: used,
-                    max_usage: self.internal_heap_stats.max_usage,
-                    total_allocated: self.internal_heap_stats.total_allocated,
-                    total_freed: self.internal_heap_stats.total_freed,
-                }
-            }
-            _ => {
-                HeapStats {
-                    region_stats,
-                    size: free + used,
-                    current_usage: used,
-                }
-            }
+            feature = "internal-heap-stats" => HeapStats {
+                region_stats,
+                size: free + used,
+                current_usage: used,
+                max_usage: self.internal_heap_stats.max_usage,
+                total_allocated: self.internal_heap_stats.total_allocated,
+                total_freed: self.internal_heap_stats.total_freed,
+            },
+            _ => HeapStats {
+                region_stats,
+                size: free + used,
+                current_usage: used,
+            },
         }
     }

@JurajSadel JurajSadel added skip-changelog No changelog modification needed skip-semver-checks labels Aug 3, 2026
@bugadani

bugadani commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

You'll need to submit this as part of #6022 otherwise we'll not be able to merge either.

@JurajSadel JurajSadel closed this Aug 3, 2026
@JurajSadel
JurajSadel deleted the fmt branch August 3, 2026 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog No changelog modification needed skip-semver-checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants