Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

minor issues with emacs support #8

Open
bhyde opened this issue May 18, 2013 · 1 comment
Open

minor issues with emacs support #8

bhyde opened this issue May 18, 2013 · 1 comment
Labels
bug

Comments

@bhyde
Copy link

bhyde commented May 18, 2013

The emacs support is delightful!

In this example...

cl-user> (log:info '(x y z) "yogurt for breakfast!")
 <INFO> [08:51:06]  (x y z) - yogurt for breakfast!
; No value
cl-user>

It seems to think yogurt is a function, but you and I know it's a dairy product.

Further, if you place the cursor on the close paren, well between the "z" and ")" and then bring up the context menu you get an ecase error. Yeah, ecase!
@7max
Copy link
Owner

7max commented May 18, 2013

Acknowledged, problem is Emacs side log4slime-highlight-log-message on Emacs needs to figure out there is no package, (backend already knows, thats why %g is printed as empty string).

Current logic says first bare word not in any kind of brackets is package.. Potential fix is to add a condition that it has to be before the "rest of the category" which is highlighted in function face.

@dkochmanski dkochmanski mentioned this issue Jan 14, 2017
Open
svetlyak40wt pushed a commit to 40ants/log4cl that referenced this issue Dec 2, 2017
@dkochmanski
cleanup: locking improvements
dcdc5ad 
- don't use bordeaux-threads package (add explicit package prefix)
- serialized appender lock is now recursive - fixes 7max#8
- add names to locks for easier identification (in case of deadlock detection)
svetlyak40wt pushed a commit to 40ants/log4cl that referenced this issue Dec 2, 2017
@scymtym
tests: add tests for 7max#8
fc63ee5
enometh pushed a commit to enometh/log4cl that referenced this issue Nov 13, 2024
@dkochmanski
cleanup: locking improvements
b22002b 
[cherry-picked from
sharplispers@dcdc5ad]

- don't use bordeaux-threads package (add explicit package prefix)
- serialized appender lock is now recursive - fixes 7max#8
- add names to locks for easier identification (in case of deadlock detection)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bhyde @7max