Re: [edk2-devel] [PATCH 2/2] DynamicTablesPkg/TableHelperLib: Enhance error handling

2023-10-11 Thread PierreGondois
Hi Jeshua, On 10/10/23 17:41, Jeshua Smith wrote: #3 is not currently used by any published code. It is a development aid, which the ObjectParser itself seems to be. Here's why I added it. Several people on our team have (not yet upstreamed) changes that resulted in additional ObjectIDs being

Re: [edk2-devel] [PATCH 2/2] DynamicTablesPkg/TableHelperLib: Enhance error handling

2023-10-10 Thread Jeshua Smith via groups.io
#3 is not currently used by any published code. It is a development aid, which the ObjectParser itself seems to be. Here's why I added it. Several people on our team have (not yet upstreamed) changes that resulted in additional ObjectIDs being added to the ObjectID enums, but without correspond

Re: [edk2-devel] [PATCH 2/2] DynamicTablesPkg/TableHelperLib: Enhance error handling

2023-10-10 Thread PierreGondois
Hello Jeshua, On 10/6/23 18:28, Jeshua Smith wrote: This patch enhances error handling and reporting in the CM ObjectParser. Specifically: 1. ObjectIDs used as array indexes are checked for being out of bounds, and if so an error message is printed before the assert. 2. An error message is p

[edk2-devel] [PATCH 2/2] DynamicTablesPkg/TableHelperLib: Enhance error handling

2023-10-06 Thread Jeshua Smith via groups.io
This patch enhances error handling and reporting in the CM ObjectParser. Specifically: 1. ObjectIDs used as array indexes are checked for being out of bounds, and if so an error message is printed before the assert. 2. An error message is printed for unsupported NameSpaceIDs. 3. Adds support for