Bug 13251 - wrong replace-in-string definition
Summary: wrong replace-in-string definition
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: emacs-color-themes (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Eugene Vlasov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-30 14:26 MSK by Alexey Voinov
Modified: 2007-11-08 14:14 MSK (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Voinov 2007-10-30 14:26:45 MSK
В color-themes.el присутствует не самая правильная конструкция:

(or (fboundp 'replace-in-string)
    (defun replace-in-string (target old new)
      (replace-regexp-in-string old new  target)))

Всё бы ничего, но это определение не совсем корректно, что приводит к конфликтам
с PlannerMode, например. Надо либо заменить здесь функцию на более правильную:

(defun replace-in-string (target old new &optional literal)
  (replace-regexp-in-string old new target nil literal))

либо заменить два вызова replace-in-string в функции define-color-theme на
вызовы replace-regexp-in-string.
Comment 1 Eugene Vlasov 2007-11-07 21:57:08 MSK
Должно быть исправлено в emacs-color-themes-6.6.0-alt2
Comment 2 Alexey Voinov 2007-11-08 14:14:46 MSK
угу. вижу. поправлено.