Re: acpiview error handling patches
Hi Zhichao,toggle quoted messageShow quoted text
Some patches in this series need reworking.
Example - For '[PATCH v3 1/8] ShellPkg/AcpiView: Extract configuration struct' & '[PATCH v3 2/8] ShellPkg/AcpiView: Declutter error counters'
I feel accessor methods offer a better design and should be retained.
Other patches in the series need a bit of rework to follow coding conventions etc.
This patch series has too many changes to the parsers. I request to put this patch series on hold as we have other patches that need to go in on priority. e.g. loop detection in PPTT table.
Since Tomas is no longer with Arm, someone from Arm will pick up repost the reworked series.
From: email@example.com <firstname.lastname@example.org> On Behalf Of Gao, Zhichao via groups.io
Sent: 31 July 2020 01:43 AM
To: email@example.com; tomas@...
Subject: Re: [edk2-devel] acpiview error handling patches
I am busy at other works recent weeks. I plan to review the patch in next two weeks. Hope it is acceptable for you.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.