Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0005136OCamlOCaml generalpublic2010-08-28 21:222010-08-30 17:16
ReporterChristophe Troestler 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version3.13.0+dev 
Target VersionFixed in Version3.12.1+dev 
Summary0005136: caml-mode: string-to-int is obsolete since emacs 22.1
DescriptionWhen compiling caml-types, I have the message

... `string-to-int' is an obsolete function (as of
    Emacs 22.1); use `string-to-number' instead.

The attached patch fixes that.
TagsNo tags attached.
Attached Filespatch file icon caml-types.patch [^] (4,613 bytes) 2010-08-28 21:22 [Show Content]

- Relationships

-  Notes
(0005644)
doligez (administrator)
2010-08-30 12:18

This is more-or-less fixed in 3.12.1+dev. The question is how to remain compatible with older versions of emacs. Currently we define caml-string-to-int to be string-to-number (if available) or string-to-int, but I don't know how to do that without triggering a warning.
(0005645)
Christophe Troestler (reporter)
2010-08-30 14:17

How about this

(if (fboundp 'string-to-number)
    (defalias 'caml-string-to-int 'string-to-number)
  (defalias 'caml-string-to-int 'string-to-int))

(I am not an elisp expert either!)
(0005646)
doligez (administrator)
2010-08-30 17:16

That seems to do the trick, thanks. I've committed it to 3.12.1+dev.

- Issue History
Date Modified Username Field Change
2010-08-28 21:22 Christophe Troestler New Issue
2010-08-28 21:22 Christophe Troestler File Added: caml-types.patch
2010-08-30 12:18 doligez Note Added: 0005644
2010-08-30 12:18 doligez Status new => feedback
2010-08-30 12:18 doligez Resolution open => fixed
2010-08-30 12:18 doligez Fixed in Version => 3.12.1+dev
2010-08-30 14:17 Christophe Troestler Note Added: 0005645
2010-08-30 17:16 doligez Note Added: 0005646
2010-08-30 17:16 doligez Status feedback => closed


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker