David Maas

Results 177 comments of David Maas

While doing some investigations for https://github.com/MochiLibraries/Biohazrd/issues/247, I found this information is retained in the `TemplateSpecializationArgsAsWritten` (for function template instantiations at least, I assume something similar exists for record template specializations.)...

Related: https://github.com/InfectedLibraries/Biohazrd/issues/108 / https://github.com/InfectedLibraries/Biohazrd/issues/156

Note: Make sure to update the documentation for `KludgeUnknownClangTypesIntoBuiltinTypesTransformation` when this is completed.

Another quirk revealed by this issue: If more than one function fails to emit a type it can break disambiguation by operator overloading. IE: ```cpp class MyClass {}; //

Honestly we should instead just fail these sorts of methods during the verification stage. Edit: We actually need to do this after the broken declaration extractor runs because it can...

> Honestly we should instead just fail these sorts of methods during the verification stage. https://github.com/InfectedLibraries/Biohazrd/issues/47

While we should definitely fail these methods during verification, I think it could be valuable to have the ability to last-minute remove a failed declaration. We already buffer the contents...

When adding new types, make sure to check if `pathogen_GetConstantValue` needs to be updated to support them. (It will support some out of the box, but not all.)

I gave this some more thought while implementing the fix for https://github.com/InfectedLibraries/Biohazrd/issues/146 I think it might be valuable for declarations to be marked for lazy emit. Essentially the declarations won't...

`TranslatedUndefinedRecord` is another good candidate for this. It's usually referenced to start with, but sometimes all references to them are removed by transformations.