On Sun, Mar 09, 2025 at 05:18:28PM +0000, Gavin Smith wrote: > Updating the test results changed "erreur" to "error". This was for > the change that introduced _cache_node_names to bulk convert all the > node names in one go. I assumed this was because the documentlanguage > wasn't set at the time the node names were converted.
Indeed, and I think that it is bad. Caching the converted node name is good, but it should be more dynamical, converted node names should be invalidated when documentlanguage changes. This is what is done in HTML for instance, and I think that it should be the same for Plaintext/Info. This does not explain why the output has changed again for C with the b1499f98293fba2c51278b7af38c3da06d8625b9 commit. There are changes related to command option setting, including for documentlanguage, but it is supposed to be a change in representation of data, but not of the data itself. It should probably be investigated, but I think that dynamically caching the nodes conversion should be implemented before. -- Pat
