Me too.

I’d be OK if to start it just listed the various options, if making the button smart is too much work. Thanks, Andrew Fish

Yes sounds good. Thanks, Andrew Fish

Mike, My main concern is there is an easy path for the developers. I don’t want to have developers have to learn new development technologies just to run a “required” tool. I’m not as concerned on how

I could not figure out how to download an installer from the links. If I go to http://uncrustify.sourceforge.net I see https://sourceforge.net/projects/uncrustify/files/ and a button to download binar

MIke, I could not figure out how to download uncrustify tool from the provided link. 99% of the people are just going to want to install the tool, not be a developer of the fork. We should have some s

+1 Thanks, Andrew Fish

Our internal repo uses EDK_TOOLS_PATH. In our case it is $(WORKSPACE)/edk2/BaseTools. We have a PACKAGES_PATH set to $(WORKSPACE)/edk2 and magic happens. So I’m thinking maybe: $(EDK_TOOLS_PATH)/Bin/G

Chao, This is probably a little out of date, but in case it helps http://vzimmer.blogspot.com/2014/11/porting-to-new-architecture.html?m=1

I had to manually accept your message. Hopefully it is OK now. Thanks, Andrew Fish

Mike, Sounds like a good plan. Thanks, Andrew Fish

Marvin, Good point. This would make the rule moot. So maybe just removing the requirement would be the easiest long term fix. Other embedded projects I know of do this too, and as you point out the co

This issue is independent of CONST. I’m not sure a coding style tool is smart enough to catch this generically? You need an understanding of C types to know if the local variable assignment is going t

On the git-blame front we might be able to build a recipe that walks past the code style and line ending commits that would could add to an FAQ or Wiki. For bonus points we could add a git command tha

Can we disable the rule that does not like the DEBUG macro? I seem to remember clang nags about some strange () usage, so maybe we would not lose that much? Thanks, Andrew Fish

Should we move to require a min Python version at some point?

Are you using Python 2.7? Try using Python 3.* It looks like array.fromstring() got renamed to array.frombytes() in Python 3.2, so I’m guessing you are using Python 2.7? Thanks, Andrew Fish

Guo, Structure is the endian of the machine running Python which might not be correct. It would be more correct to use LittleEndianStructure vs Structure. This is probably good practice like using “<“

If we add QuickSort() API to the BaseLib can we not just port the existing MdeModulePkg/SortLib to use QuickSort() in the implementation? Or is there some other way to add the new thing in a backward

Sorry the patches stalled out. I need to push them…. Thanks, Andrew Fish