[Bug 207781] Review Request: libmatheval - parse and evaluate symbolic expressions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatheval - parse and evaluate symbolic expressions


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=207781





------- Additional Comments From ed@xxxxxxx  2006-09-23 22:24 EST -------
Hi Jason, thatanks for taking a look.  I spent some time today digging 
into the "make check" errors and it turns out that there are some 
deprecated guile bits which are:

  SCM_STRINGP is deprecated.  Use scm_is_string instead.
  SCM_LENGTH is deprecated.  Use scm_c_string_length instead, 
    for example, or see the manual.
  SCM_STRING_LENGTH is deprecated.  Use scm_c_string_length instead.
  SCM_CHARS is deprecated.  See the manual for alternatives.
  SCM_STRING_CHARS is deprecated.  See the manual for alternatives.

The interesting thing is that these deprecated bits *ONLY* appear
in the the file "libmatheval-1.1.3/tests/matheval.c" and nowhere else 
in the actual libmatheval code.

So it appears that there are some deprecated bits only in the testing 
code that then triggers some warnings that then causes one of the tests 
(test number 5) to "fail" (although the test actually works, it just 
emits a bunch of these warnings so that the test script *thinks* that 
it has failed).

In any case, I'll report the deprecated bits in the testing code to 
the upstream folks.  But I see no reason to do anything else since 
there appear to be no other deprecated (or otherwise broken) bits in 
any of the actual library code.

Here's an update that includes the BR: texinfo

  http://mitgcm.org/eh3/fedora_misc/libmatheval-1.1.3-4.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]