Skip to content

Update instruction files#1456

Draft
p-bond wants to merge 1 commit into
dev_sprint_25_2from
feature/VPAAMP-4_update_using_coding_guidelines
Draft

Update instruction files#1456
p-bond wants to merge 1 commit into
dev_sprint_25_2from
feature/VPAAMP-4_update_using_coding_guidelines

Conversation

@p-bond
Copy link
Copy Markdown
Contributor

@p-bond p-bond commented May 15, 2026

Full detail on method in ticket, including derived coding guidelines.

| `AAMPLOG_TRACE` | Trace | Development and triage-level detail; verbose output for deep debugging. |
| `AAMPLOG_INFO` | Info | Informative and debug messages; especially useful during tune operations. |
| `AAMPLOG_WARN` | Warn | Recoverable warnings or important runtime conditions worth noting. |
| `AAMPLOG_ERR` | Error | Severe or unexpected conditions that warrant investigation. |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This list is missing AAMPLOG_MIL. Originally this level did not exist in AAMP and WARN was used for log lines that were important enough to be printed in the log by default, even though they were not warnings. Now we should be using AAMPLOG_MIL instead for those lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants