Skip to content

Commit

Permalink
LibSpiro version 0.5
Browse files Browse the repository at this point in the history
Collective fixes from tagpoint v0.3 (20150131) up to now v0.5 (20150702).
Closes issues #11 seen on https://github.com/fontforge/libspiro/issues
due to missing file. Coverity scan caught resource leak (added free().
  • Loading branch information
JoesCat committed Jul 2, 2015
1 parent 044e109 commit edfab0f
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 6 deletions.
10 changes: 8 additions & 2 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,9 +1,15 @@
- 2015-Jul-1
- 2015-Jul-02
* Libspiro Version 0.5.20150702
* Important bug fix issue #11 (missing file not included in 0v4.
* 2 Minor bug fixes caught on Coverity scan, free() and if c=3

- 2015-Jul-01
* Libspiro Version 0.4.20150701
* Re-edit lib and tests to be more accommodating of older compilers.
* Many changes to call-test.c to run on bigger variety of platforms,
and some minor configure.ac tweaks done to resolve issues #9, #10.
as seen on https://github.com/fontforge/libspiro/issues (closed).
as seen on https://github.com/fontforge/libspiro/issues and:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=789012
* Added developer/install test for cyclic spiros (without {} or z).
* Verify libspiro output data is correct for test curves {0,1,2,4}.
* User can overide default configure flags with LS_CFLAGS and LS_LIB
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Spiro ![](spiral32.png)
# Spiro ![](spiral32.png) [![Coverity Scan Build Status](https://scan.coverity.com/projects/794/badge.svg?flat=1)](https://scan.coverity.com/projects/794)

## Introduction

Expand Down
4 changes: 2 additions & 2 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dnl Process this file with autoconf to produce a configure script.
AC_PREREQ([2.61])
#-------------------------------------------
# PackageTimestamp version
m4_define([spiro_package_stamp], [20150701])
m4_define([spiro_package_stamp], [20150702])
#-------------------------------------------
# Making point releases:
# spiro_major_version += 0;
Expand All @@ -21,7 +21,7 @@ m4_define([spiro_package_stamp], [20150701])
# spiro_minor_version = 0;
#
m4_define([spiro_major_version], [0])
m4_define([spiro_minor_version], [4])
m4_define([spiro_minor_version], [5])
m4_define([spiro_version],
[spiro_major_version.spiro_minor_version.spiro_package_stamp])
m4_define([spiro_info],
Expand Down
3 changes: 2 additions & 1 deletion tests/call-test.c
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ int test_curve(int c) {
if ( c==0 ) rsp = verify_rs0;
else if ( c==1 ) rsp = verify_rs1;
else if ( c==2 ) rsp = verify_rs2;
else if ( c==3 ) rsp = NULL; /* expecting failure to converge */
/* else if ( c==3 ) rsp = NULL; expecting failure to converge */
else rsp = verify_rs4;

/* Quick visual check shows X,Y knots match with each pathN[] */
Expand All @@ -253,6 +253,7 @@ int test_curve(int c) {
(fabs(segs[i].seg_th - rsp[i].th) > 1e-5) ) {
printf("FAIL\nerror found with run_spiro() data. Results are not the same.\n");
printf("expected line %d x=%f y=%f t=%c bend=%f ch=%f th=%f \n",i,spiro[i].x,spiro[i].y,spiro[i].ty,rsp[i].b,rsp[i].ch,rsp[i].th);
free(segs);
return -2;
} else
printf("PASS\n");
Expand Down
2 changes: 2 additions & 0 deletions tests/call-test4.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
#define DO_CALL_TEST4 1
#include "call-test.c"

0 comments on commit edfab0f

Please sign in to comment.