Laszlo, Thank you for the pointers to the dependent header files.
I agree that main API header files should include any headers that they depend upon and not require the developer to search out the dependencies. Most modern pre-processors and compilers have optimizations to mitigate reading the same header file multiple times. This invalidates the old rationale for policies that don't allow header files to include other header files. Now, I need to find out why my search tool didn't find those headers. ☹ In gratitude, Daryl McDaniel The Year of the Justifiably Defensive Lobster. The Season of the fraternizing CEOs. -----Original Message----- From: edk2-devel <[email protected]> On Behalf Of Laszlo Ersek Sent: Wednesday, August 1, 2018 2:56 PM To: [email protected]; [email protected] Subject: Re: [edk2] Undefined types referenced in MdePkg On 08/01/18 19:49, [email protected] wrote: > When including MdePkg/Include/Library/HobLib.h in my project I get > errors indicating that the following types are undefined: > > * EFI_RESOURCE_TYPE, lines 206 and 231 > * EFI_RESOURCE_ATTRIBUTE_TYPE, lines 207 and 232 > > > > A similar thing is happening with MdePkg/Include/Protocol/FirmwareVolume2.h. > > * EFI_FV_FILE_ATTRIBUTES, lines 309, 443, 612 > > > > I updated my EDK II tree then searched the entire tree for these > types. They only show up in the locations mentioned above; never defined. > > > > Am I missing something or are these really undefined? > > > > I don't want to just disable these errors since that could hide a > bigger problem. IMO, the header files you mention should include the following files, respectively: - MdePkg/Include/Library/HobLib.h --> MdePkg/Include/Pi/PiHob.h - MdePkg/Include/Protocol/FirmwareVolume2.h --> MdePkg/Include/Pi/PiFirmwareVolume.h I'm firmly in the camp that believes that developers should not have to hunt down themselves the dependencies of the main API header file that they are actually interested in. Thanks Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
