evelez7 wrote:

> This is a good start, but I thin we'll want some more tests, and probably 
> some unit test coverage. Unittesting is especially nice, since I believe this 
> backend doesn't need any of the asset files, right?

Yeah, this doesn't need assets, so I'll be able to call things directly for 
unit tests.

> As for whether this should follow the pattern of YAML or HTML ... I'm 
> conflicted for the same reasons you are. But maybe there's a better way for 
> us to generate the filename that doesn't need the USR? like could we get the 
> mangled name and use that? I'd hope those would be different enough to not 
> conflict. Alternatively, maybe we should look at a way we can merge all the 
> related docinfo together at an earlier step.

I'll investigate this. Although right off the bat, the names in the Infos 
aren't mangled. `Name` and `FullName` are the same for both records, which is 
unfortunate.

https://github.com/llvm/llvm-project/pull/142483
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to