vrahane commented on code in PR #3400:
URL: https://github.com/apache/mynewt-core/pull/3400#discussion_r2076162700


##########
sys/log/full/syscfg.yml:
##########
@@ -196,6 +202,10 @@ syscfg.defs:
         restrictions:
             - LOG_FCB_BOOKMARKS
             - LOG_FCB
+    LOG_MAX_TRAILER_LEN:
+        description: >
+            Maximum length of trailer that can be appended to a log entry
+        value: "LF_MAX_ALIGN * 3"

Review Comment:
   If the max alignment changes, the max trailer length is going to get 
affected because alignment plays a big role in fitting the trailer, the 
implementation of a trailer can be a TLV, a reverse TLV or static data but in 
order for it to be read successfully from the end, alignment matters. I would 
prefer to keep it this way if you don't feel strongly.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to