Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

target_info: print ram region again #224

Merged
merged 1 commit into from
Jun 9, 2021
Merged

target_info: print ram region again #224

merged 1 commit into from
Jun 9, 2021

Conversation

Lotterleben
Copy link

after the snapshot test rebase I noticed we're not printing the RAM region anymore. I'm not sure if I'm misremembering that that was on purpose. If I am misremembering, this PR fixes that. If I'm not misremembering, it leaves a paper trail.

@Lotterleben Lotterleben requested a review from japaric June 7, 2021 17:12
Copy link
Member

@japaric japaric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. if's debug statement so not much harm in having it since it's not printed by default

@Lotterleben
Copy link
Author

bors r+

@bors bors bot merged commit fbf3810 into main Jun 9, 2021
@bors bors bot deleted the print-ram-region branch June 9, 2021 08:32
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants