search

zbelial / lspce

LSP Client for Emacs implemented as a module using rust.
GNU General Public License v3.0
154 stars 11 forks source link

add build options #34

Closed mopemope closed 4 months ago

mopemope commented 4 months ago

Added optimal build options using cargo wizard.

zbelial commented 4 months ago

Hi @mopemope , thanks. I have a question about panic = "abort" though, if we set this, I'm not sure whether a panic will make Emacs exit or not? If it will, we may lose the chance to see what causes panic, right?

mopemope commented 4 months ago

Hi. @zbelial Yes, you're right. I'll fix that since it's set to abort for speed.

zbelial commented 4 months ago

Thanks. I just merged. But I'm thinking about deleting the lld flag later for these reasons:

  1. Users have to install lld, and
  2. I read from somewhere that lld is not fully supported yet. Here is a issue from rust-lang.
  3. It just makes linking faster, so IMO it's not that important as the other changes.