lambda-list: Improve error message in c-compiler

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

lambda-list: enforce unique names among required parameters

Signal an error, if two required parameters have the same name.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cosmetic: undef unused macro

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

flet/labels: make same function name error message more descriptive

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

LET: signal an error when multiple bindings of the same name occur

This situation is unspecified by ANSI spec – also hard to find
situation, when such construct wouldn't be an error.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

c_leta: don't compute whole list length for optimization

This is interpreter performance tweak. Behavior stays the same.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

flet/labels: signal an error if same name is used more than once

This behavior is unspecified by the ANSI spec, so we are free to do
that – can't imagine valid usecase of such blocks.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

asdf: update to version 3.1.5.4

This version has fixed make-image and uses run-program.
ECL backend updates possible thanks to help of Francois-Rene Rideau.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

rt: add deftask* and untabify sources

deftask* evaluates result agument.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

tests: add test-ansi suite stub

For now contains readtable case tests.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: add recent changes

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

tests: mop-dependents: remove typo (?)

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: stylistic cleanup provided by Matthew Mondor

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

tests: fix broken testing framework

This still needs refinements, but works.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

tests: fix wrong assumption on assert

Tests using assert assumed it returns (values) — no value, but it
returns nil if assertion is true. Add it to deftests.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

encodings: fix misplaced function reference

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: update document

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

external-process: rename struct member to error-stream

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: refine documentation for external-process

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: update run-program documentation

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

run-program: allow further customization of process communication

Implement key parameters `if-input-does-not-exist', `if-error-exists'
(for pathnames) and `external-format' for both pathnames and streams.

Fixes #107.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: run-program: correct mistake, add pathname

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: document run-program and external-process

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

system: set streams to nil

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

rt: fix mixied package lock test

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

tests: remove unused/outdated/unrelated tests

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: asdf: update api information

Corrections provided by Fare Rideau

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: cosmetic: II.5 line wrap + typos

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: II.4.1 fix typo

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: verify and update II.3 (ffi) section

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: run-program: add limitations section

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: update asdf manual url

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

run-program: error stream default value is :output

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: fix broken section tag

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

trace: improve warning message

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: I.11.1C: add FFI package

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: fix typo at 1.5.3

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: mention single-float and double-float built-in-classes in 1.4.1

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: fix numbering of "Standard" section to match CLHS

Fixes #98.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: verify "1. Overview", wrap some lines to 75 characters

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: add windows 8 to tested platforms

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: update autoconf section outputs

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: update history section

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: remove information about clos being optional

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

Revert "cmp: ffi: cast `ecl_make_foreign_data' 3rd argument to *(void **)"

This reverts commit ab50a55fb183d750b3fa356e32bf112c563bed17.

We can't cast every 3rd argument to *(void **) - this breaks *things*.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

logical-pathname-translations: signal error if pathname not defined

If logical pathname wasn't defined yet, signal type error. Fixes #102.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

doc: add documentation as doc subdirectory

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: add section 'API changes'

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

run-program: make invalid handler errorr message more readable

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

ffi: remove package nickname 'uffi'

Our ffi isn't uffi 2.0 compatible and most systems wants this, when
they say (require 'uffi). Also if system has asdf dependency on
quicklisp) we have package name error.

We'll depend with UFFI 2.0 compability on cffi-uffi-compat, what is
the saniest approach. Some ECL specific patches will land there soon
to improve performance - no internal FFI api will change. It's just
nick removal.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: update on common changes

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

format: fix bug for empty width and fdigits

When neither w and d weren't set, then k was ignored.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: ffi: cast `ecl_make_foreign_data' 3rd argument to *(void **)

Some compilers doesn't allow putting function pointers when parameter
type is declared (void *), but it's necessary if we want to use
callbacks. Fixes #99.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: add update about unique function names

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: search for tag only when name is unique

fasl and program init function names are always the same, don't search
for tag in binary files for this two targets.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: use unique names for inititalization functions of libraries

Fixes #74.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: embed unique tag with init function name in each exported module

This is necessary if we want to retrieve function name from compiled
binary, what is necessary if we want to use unique names for this
function. This is necessary for fix of issue #41.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: make init-function-name consistant with compute-init-name

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cmp: remove redundant compute-main-name function

Same functionality is provided by compute-init-name

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

buildsystem: add src/*cmp/*.lsp to ecl dependencies

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

compiler: bind lexically *readtable* when file is compiled

Both byte and C compilers didn't bind lexically *readtable*, what lead
to pollution of other compiled packages. Fixes #95 and #59.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

reader: readtable-case works approprietly for :invert and others

Reader had problems with converting case. Firstfully, it did convert
case for character literals. Second, it didn't convert characters for
some specific symbol cases (especially when some were escaped). Fixes

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cosmetic: libffi updates this file on each compilation

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

changelog: add new float specializers to changelog

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

cosmetic: short-float: fix inconsistencies with short-float

Altough short-float is now implemented as single-float, there are
clear signs that there were attempts / was separate implementation of
that type with partly-abandoned efforts. This patch shapes a bit code
in predlib.lsp and adds comments in appropriate places, that this
files needs to be modified, if such separate implementation occurs.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

clos: add single-float and double-float to built-in class hierarchy

Some libraries want methods to specialize on double-float and ECL
fails on such attempts. Fixes #96.

Signed-off-by: Daniel Kochmański <daniel@turtleware.eu>

README: change filename

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

si_reset_mergins: move function declaration to external.h

Build with cxx compiler should work now. Symbol was put in internal.h,
while it is accessed from outside.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

changelog: update

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

cmp: mapcar: evaluate function only once

When function is a closure, all lexical environment were reinitialized
on each function call, because it was expanded multiple times. Do it
only once and use result in funcall. Fixes #17.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

newcmp: deduplicate cmpmap.lsp

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

clos: fix `print-object' for non- standard-object's

Previous commit overgeneralized. Handle just one particular corner
situation, when print-object leaks from standard-object to builtin
class T.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

clos: fix `print-object' specialized on T (infinite recurrence)

Corner case. When user despite spec defines method `print-object'
specialized on standard-object and calls call-next-method,
`print-object' specialized on T called `write', which triggers
original `print-object' specialized on `standard-object'.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

autotools, gc: refine aclocal.m4, refine GC detection, remove hack

Remove `ifdef' against GC version with hack - if GC too old use
included one. Also set default stack direction to `down' (on
cross-compilation) and fix cross-compilation of GMP. Fixes #87.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

msvc: rename labels `INT' and `FLOAT' to fix build with VS2013 CE

Patch provided by community members lll and Dinosky. Fixes #7.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

signals: don't whitelist suspend/resume signals if GC below 7.4

GC 7.2 doesn't export `GC_get_thr_restart_signal`, so we can't
whitelist it. Gives warning about old GC.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

cross-compile: set default stack direction to down

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

buildsystem: fix builds with flag --with-clos-streams=no

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

format.d: accept t_string as ~? argument

Only when UNICODE is enabled obviously.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

gmp: fix cross-compilation problem when passing --host

https://gmplib.org/list-archives/gmp-bugs/2012-January/002511.html

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

libatomic_ops: bump to 7.5.0

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

changelog: add recent (not crucial) changes

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

CLOS: remove dead code and conditionals for non-clos builds

Builds without CLOS weren't possible for long time, yet a lot of dead
code for non-clos builds was spread across sources. This commit
removes all ifdefs and dead blocks from codebase.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

with-unique-names: simplify extension and fix typo

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

cosmetic: indent-tabs-mode typo

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

cosmetic: lsp: setq emacs local variable indent-tabs-mode to nil

This changes local variable indent-tabs-mode to nil for all *.lsp
files in src/lsp directory.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

define-compiler-macro: clhs compliance

This change is incorporated from CLASP. Fixes #82.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

signals: Don't block GC suspend/resume signals on interrupt thread

Blocking all signals except default interrupt caused OpenBSD broken
multithread builds, because of GC semaphore lockup waiting, untill all
threads are stopped. Fixes #71.

Signed-off-by: jack <jack@openbsd.my.domain>

changelog: add gmp fix

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

bignums: don't collect gmp internal memory

GMP manual at https://gmplib.org/manual/Custom-Allocation.html sates,
taht "GMP may use allocated blocks to hold pointers to other allocated
blocks. This will limit the assumptions a conservative garbage
collection scheme can make.", therefore we won't collect it's internal
blocks. Fixes #58.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

ecl_stream_to_handle: fix problem when passing stream

Function for some types of streams without handler (not all) was
calling internal-error stopping whole implementation instead of
returning `INVALID_HANDLER'.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

run-program: handle `error' parameter value `:output'

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

changelog: run-program works under cygwin

Cygwin fork/exec doesn't seem broken anymore. Fixes #52.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

run-program: Major function refactor

A lot of code was duplicated due to creating descriptors for output,
input and error streams. It is reduced by extracting function
`create_descriptor'.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

run-program: Export error stream symbol

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

run-program: Implement `stderr` handling as stream and as pathname

This makes `stderr` same class citizen as `stdout` and `stdin`. In
order to make it work structure process has been changed to be able to
handle stream_error and index of `exit-code` and `status` are
1+. Access to stream is possible via last value of run-program, which
returns process itself.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

run-program: fix order of if/else clauses (sanity patch)

Order of checking the same conditions in input and output was slighly
different without an obvious reason. Fixed that, so code is easier to
read.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

changelog: add GMP-6.0.0 bug to known issues

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

stack-overflow: improve condition message

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>

stack: Protect user from resizing stack above maximum limit

If `rlimit' is available (ie on linux), then don't extend stack above
maximum value. If we don't know limit or limit is `infinity', then
there is no protection and user extends heap on his own risk.

Signed-off-by: Daniel Kochmański <dkochmanski@turtle-solutions.eu>