-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
Fix lexical-dynamic warnings #284
Conversation
@@ -1653,7 +1653,7 @@ form is what is returned for PATH." | |||
(file-readable-p path)) | |||
(setq path (hpath:call | |||
(lambda (path non-exist) | |||
(let (modifier | |||
(let ((modifier nil) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rswgnu modifier
is strange. Never set and used in call to two later concat
calls. Code might be possible to clean up more so leaving this for you to look into if modifier
can be removed and the concat
calls simplified?
@@ -1395,8 +1395,7 @@ new outlines is also set to this new value." | |||
(- sep-len-increase) ?\ ))) | |||
(while (re-search-forward indent nil t) | |||
(delete-region | |||
(+ (match-beginning 0) 2) (match-end 0))))))) | |||
pos) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pos does not look like it is used so removing it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right that it is not really used but it is sent in a call about 2 lines down but then the value is ignored in the called function as well. Let's push your updates and then I'll finish the changes here after.
@@ -1395,8 +1395,7 @@ new outlines is also set to this new value." | |||
(- sep-len-increase) ?\ ))) | |||
(while (re-search-forward indent nil t) | |||
(delete-region | |||
(+ (match-beginning 0) 2) (match-end 0))))))) | |||
pos) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right that it is not really used but it is sent in a call about 2 lines down but then the value is ignored in the called function as well. Let's push your updates and then I'll finish the changes here after.
972ed4d
to
daa3031
Compare
What
Fix lexical-dynamic warnings.
Why