search

svaante / dape

Debug Adapter Protocol for Emacs
GNU General Public License v3.0
448 stars 25 forks source link

Missing dape-read-pid #75

Closed yoav-za closed 5 months ago

yoav-za commented 5 months ago

dape-read-pid got removed on acc56cb89ce0fa76c9a43c5fe6d4cbfd022cc3ec. I believe it happened by accident, as dape-read-pid is unrelated to this commit's topic.

svaante commented 5 months ago

Hey, was not an mistake but the commit history makes is seam that way.

As it was not used in any batteries included adapter I thought it made sense to remove it as it is one less piece of code to maintain.

yoav-za commented 5 months ago

Thanks for your response. Although one can implement dape-read-pid themselves, since without this function there is no an easy way of dynamically attaching a debugger to a running process (when debugging C, C++, Rust, etc.), which is the whole purpose of a debugger, the solution IMO is incomplete without it.