joaotavora / sly

Sylvester the Cat's Common Lisp IDE
1.27k stars 144 forks source link

Equivalent to slime-profile-*? #306

Open Ambrevar opened 4 years ago

Ambrevar commented 4 years ago

Is it missing from SLY? If so, is there any plan to add it? Thanks!

joaotavora commented 4 years ago

I don't know what slime-profile is, I only now what slime-sprof is. It's integration with SBCL's profiler, not a portable contrib, so I took it out of SLY.

However, you can make third party contribs for SLY, so you can probably just take slime-sprof.el's code (and its dependencies) and create a new contrib based on https://github.com/joaotavora/sly-hello-world . I just updated the instructions for doing so.

joaotavora commented 4 years ago

@Ambrevar I'm curious, are you going to attempt the new contrib? Or do you want someone else to try?

Ambrevar commented 4 years ago

Maybe some day, but chances are low to be honest :) Better if someone else chimes in! :)

joaotavora commented 4 years ago

Maybe some day, but chances are low to be honest :)

Then why did you close the issue? Let's re-open it.

Ambrevar commented 4 years ago

Oh, I assumed you would not want an open issue about this, my bad :)

joaotavora commented 4 years ago

Well, I'll normaly only close issues if the need for them is solved. You opened the issue because you need this functionality in SLY, I presume.

port19x commented 8 months ago

+1 for the profiler Should I work on a PR for this?

joaotavora commented 8 months ago

@port19x better a contrib: as mentioned here https://github.com/joaotavora/sly/issues/306#issuecomment-590991488

port19x commented 7 months ago

Looks like I won't work on this in the foreseeable future, sry