)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"21cad93688e4c42f9f7dc5009120e42adfe19ec5","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Currently, we are using offset of xListEnd.pxPrevious in List_t for"},{"line_number":10,"context_line":"list_next_offset and offset of pxPrevious in ListItem_t for"},{"line_number":11,"context_line":"list_elem_next_offset. This is confusing. Fix this."},{"line_number":12,"context_line":"As the ralated lists are doubly linked lists, only iteration order is"},{"line_number":13,"context_line":"changed without breaking functionality."},{"line_number":14,"context_line":"Also document those offsets."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"166c052a_f97ac994","line":12,"updated":"2025-04-16 21:41:30.000000000","message":"s/ralated/related/","commit_id":"d7bf1ec2ac328107dbe223e2bf1f18a769da3c8a"},{"author":{"_account_id":1002399,"name":"Chien Wong","email":"m@xv97.com","username":"ivq"},"change_message_id":"6e0459e94a7f930618cec8c6893891b676dd452e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, we are using offset of xListEnd.pxPrevious in List_t for"},{"line_number":10,"context_line":"list_next_offset and offset of pxPrevious in ListItem_t for"},{"line_number":11,"context_line":"list_elem_next_offset. This is confusing. Fix this."},{"line_number":12,"context_line":"As the ralated lists are doubly linked lists, only iteration order is"},{"line_number":13,"context_line":"changed without breaking functionality."},{"line_number":14,"context_line":"Also document those offsets."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9b194911_0404e577","line":12,"in_reply_to":"166c052a_f97ac994","updated":"2025-04-17 03:09:25.000000000","message":"Done","commit_id":"d7bf1ec2ac328107dbe223e2bf1f18a769da3c8a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"21cad93688e4c42f9f7dc5009120e42adfe19ec5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9d66014_bbfcfcd6","updated":"2025-04-16 21:41:30.000000000","message":"Thanks for the patch.\nJust one minor typo in the commit message, please fix it. I think you can use the web interface of gerrit for this simple fix.\n\nThis part of code has not received any update since it was submitted in 2012!\nI\u0027m not familiar with FreeRTOS, so I will left this patch pending for few weeks in the hope someone else reviews it, then I will merge it.\n\nThese hardcoded offset are ugly because depend on rtos version, target type, compile options (with `configUSE_LIST_DATA_INTEGRITY_CHECK_BYTES\u003d1` all offsets are different), ...\n\nIn zephyr rtos are defined dummy symbols, like `_kernel_thread_info_offsets`, that cleanly provide the needed offsets without increasing the binary size.\nhttps://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/debug/thread_info.c\n\nIf you are active on FreeRTOS, please consider to improve the rtos in a similar way for better debugger compatibility.","commit_id":"d7bf1ec2ac328107dbe223e2bf1f18a769da3c8a"},{"author":{"_account_id":1002399,"name":"Chien Wong","email":"m@xv97.com","username":"ivq"},"change_message_id":"6e0459e94a7f930618cec8c6893891b676dd452e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0fb3f356_4d4848e1","in_reply_to":"d9d66014_bbfcfcd6","updated":"2025-04-17 03:09:25.000000000","message":"Good point! Would try to add such support if I have the time.","commit_id":"d7bf1ec2ac328107dbe223e2bf1f18a769da3c8a"}]}