[PATCH] D139774: [libclang] Add API to set temporary directory location
    Igor Kushnir via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Feb  1 08:01:27 PST 2023
    
    
  
vedgy added a comment.
Perhaps the struct should contain the value of `CINDEX_VERSION_MINOR` instead of the `sizeof`? This way, libclang can change the meaning of a struct member without changing the size of the struct. For example, replace `PreambleStoragePath` with `TemporaryDirectoryPath`. Such a change could even be quiet, because setting the more general temporary directory path would likely be welcome or at least acceptable to the API users.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139774/new/
https://reviews.llvm.org/D139774
    
    
More information about the cfe-commits
mailing list