goldsteinn wrote:

> > All things considered, I think requiring `vc` is a lot less intrusive than 
> > essentially our own `vc` wrapper....
> 
> Not sure what you mean exactly by a wrapper, if git/diff logic can be 
> abstracted away - that seems a net gain. If ugly/fragile logic is required to 
> do that - it's debatable and may not be worth it (all things considered).

Well the wrapped is to hide the ugly git/diff stuff.

> 
> Anyway, I was only mentioning that we could consider abstracting away the 
> logic - if it's not practical, there is no need to go into details discussing 
> it, although I did mail the emacs-devel mailing list to check if this might 
> be supported:

My preference would be the get this in as is.


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

Reply via email to