I am reviewing your previous patch "Add security checks in the Acpiview table
parsers". That takes times and I didn't finish reviewing them yet.
You do a pretty good separation. That makes the review more easier. I would
give some comments on your previous one 10/10 patches to help your further w
Reviewed-by: Jaben Carsey
Thanks
-Jaben
> -Original Message-
> From: Krzysztof Koch [mailto:krzysztof.k...@arm.com]
> Sent: Thursday, July 18, 2019 3:05 AM
> To: devel@edk2.groups.io
> Cc: Carsey, Jaben ; Ni, Ray ;
> Gao, Zhichao ; sami.muja...@arm.com;
> matteo.carl...@arm.com; n...@arm
Reviewed-by: Sami Mujawar
-Original Message-
From: Krzysztof Koch
Sent: 18 July 2019 11:05 AM
To: devel@edk2.groups.io
Cc: jaben.car...@intel.com; ray...@intel.com; zhichao@intel.com; Sami
Mujawar ; Matteo Carlini ; nd
Subject: [PATCH v1 0/8] Remove redundant forward declaration
This patch series removes forward static function declarations in the
acpiview table parsers. After repositioning blocks of code, these forward
declarations are no longer needed and they increase the code size.
What is more, some of the existing ACPI table parsers don't have these
duplicate declar
