Bug 13251

Summary: wrong replace-in-string definition
Product: Sisyphus Reporter: Alexey Voinov <voins>
Component: emacs-color-themesAssignee: Eugene Vlasov <eugvv>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2    
Version: unstable   
Hardware: all   
OS: Linux   

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
угу. вижу. поправлено.