cdominik / cdlatex

Fast input methods to enter LaTeX environments and math with GNU Emacs
231 stars 48 forks source link

Remove duplicate definition of cdlatex-mode-map #45

Closed longminwang closed 2 years ago

dankessler commented 2 years ago

Without this fix, cdlatex-mode is effectively broken for me, i.e., none of the usual keybindings work. I suspect this is due to some property of elisp evaluation I don't fully understand, but if I activate cdlatex-mode and then inspect the value of cdlatex-mode-map, it's an entirely empty sparse keymap, so it seems that this redundant (but earlier) definition is masking the second, fulsome definition that comes later. Commenting out the lines that this PR removes fixes the problem for me.

cdominik commented 2 years ago

Thank you for the report, should be fixed now.

Carsten

On Mon, Aug 15, 2022 at 1:50 AM longminwang @.***> wrote:


You can view, comment on, or merge this pull request online at:

https://github.com/cdominik/cdlatex/pull/45 Commit Summary

File Changes

(1 file https://github.com/cdominik/cdlatex/pull/45/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/cdominik/cdlatex/pull/45, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAIABJ7RVPNEKHW5S6MLALVZGA23ANCNFSM56QVAYCA . You are receiving this because you are subscribed to this thread.Message ID: @.***>