catacomb-python
4 years agoPort to Python 3.
Mark Wooding [Tue, 22 Oct 2019 18:12:28 +0000 (19:12 +0100)]
Port to Python 3.

Most of this is `#ifdef ...' ... `#endif' noise, with a few tweaks
thrown in.

Some notes on specific parts of the port.

  * buffer.c: The Python 3 buffer protocol is completely different.
    Read buffers work differently, but aren't problematic; write buffers
    can now be held open for an extended period, so we need the locking
    machinery that was added recently.

  * catacomb.c: Module initialization has changed, but isn't a great
    deal more difficult.

  * catacomb/__init__.py, catacomb/pwsafe.py, pock, pwsafe: There are a
    number of places which need language-version switches, but none of
    them is especially interesting.  This diff is noisier than it should
    be because I couldn't adjust the indentation in advance.

  * mp.c: With the abolition of a separate fixnum type, `mp_frompylong'
    needed to express the fast path from a fixnum in a different way.

  * pwsafe: The hacking to alter the error-handling behaviour associated
    with the `stdout' stream is deeply unpleasant.  Sorry.

  * pyke/pyke.h: Most of the porting work happens here, with alternative
    definitions for the various macros introduced earlier.

  * .gitignore: Ignore Python 3 `__pycache__/' turds.  Python 3 leaves
    its pre-tokenized files in `__pycache__/' directories, which somehow
    manage to be much more objectionable than the loose Python 2 `*.pyc'
    files.  Ignore these.

  * debian/: Add the necessary things to build a Python 3 extension
    package.

4 years ago*.[ch]: Some preparatory reformatting for the Python 3 porting.
Mark Wooding [Tue, 22 Oct 2019 17:50:19 +0000 (18:50 +0100)]
*.[ch]: Some preparatory reformatting for the Python 3 porting.

Nothing very complicated or profound.  These are intended to make the
upcoming diff easier to check for stupid mistakes.  A lot of this is
whitespace change that disappears in `git show -w'.

4 years agofield.c, mp.c: Implement the `nb_index' conversion.
Mark Wooding [Mon, 18 Nov 2019 14:52:54 +0000 (14:52 +0000)]
field.c, mp.c: Implement the `nb_index' conversion.

This is Yet Another Integer Conversion added in 2.5, for the purpose of
/implicit/ conversions (since strings and other strange things will
convert explicitly to integers).

It's significant because the `bin' builtin (conversion to base-2 string,
added in 2.6) works by applying the `index' conversion rather than
having a special-purpose method like `hex' and `oct'.  (Indeed, Python 3
abolishes these methods, and `hex' and `oct' convert via `index' too.)

So add conversions somewhat judiciously.

4 years agorand.c: Cope with large `range' arguments.
Mark Wooding [Fri, 15 Nov 2019 18:26:57 +0000 (18:26 +0000)]
rand.c: Cope with large `range' arguments.

Python 3 will report large numbers as being `int' values, and
`PyInt_AS_LONG' will return nonsense in this case.

4 years agorand.c: Some gratuitous reformatting.
Mark Wooding [Fri, 15 Nov 2019 18:28:35 +0000 (18:28 +0000)]
rand.c: Some gratuitous reformatting.

4 years agocatacomb/__init__.py, mp.c: Remove `L' suffix from `repr' output.
Mark Wooding [Thu, 21 Nov 2019 22:44:15 +0000 (22:44 +0000)]
catacomb/__init__.py, mp.c: Remove `L' suffix from `repr' output.

Python 3 doesn't produce an `L', and it doesn't really help anything.

4 years agomp.c, catacomb/__init__.py, pyke/: Fix mixed-mode arithmetic involving `float'.
Mark Wooding [Tue, 22 Oct 2019 17:31:57 +0000 (18:31 +0100)]
mp.c, catacomb/__init__.py, pyke/: Fix mixed-mode arithmetic involving `float'.

This is a bit embarrassing.

>>> import catacomb as C
>>> x = C.MP(5)
>>> x == 5.1
True
>>> x < 5.1
False
>>> r = x/2
>>> r
5/2
>>> r == 2
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: an integer is required
>>> r == 2.5
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: an integer is required
>>> r*1.0
5/2
>>> r*1.1
5/2
>>> r*2.0
5
>>> r*2.5
5

Fix this nonsense.

  * Change the `obvious' arithmetic operators so that they notice that
    one of the operands is a float.  Handle this by converting to a
    Python bignum and letting Python handle the arithmetic.  The result
    is a float, which seems sensible inexact contagion.

  * Introduce a rich-comparison method which also detects a float
    operand and hands off to Python.  Python seems to get this right,
    comparing the float to the bignum in its full precision, so that's a
    win.

  * Also, modify the `IntRat' code to apply inexact contagion in the
    same way.  Comparisons may be imperfect here, but that's
    surprisingly hard to get right.

The new results:

>>> import catacomb as C
>>> x = C.MP(5)
>>> x == 5.1
False
>>> x < 5.1
True
>>> r = x/2
>>> r
5/2
>>> r == 2
False
>>> r == 2.5
True
>>> r*1.0
2.5
>>> r*1.1
2.75
>>> r*2.0
5.0
>>> r*2.5
6.25

4 years agoec.c, field.c: Remove redundant `FOO_pyrichcompare' methods.
Mark Wooding [Tue, 22 Oct 2019 16:53:44 +0000 (17:53 +0100)]
ec.c, field.c: Remove redundant `FOO_pyrichcompare' methods.

The abstract base classes already have the necessary comparison methods,
so there's nothing to be gained by repeating the same methods on
subclasses.  (Indeed, Python might pointlessly allocate more memory for
the corresponding Python-level descriptors, but I haven't checked.)  But
there is something to be lost: Python 3 explicitly clobbers the
`__hash__' methods of subclasses which declare new equality methods.  We
could add duplicates of the hash methods throughout the subclass graph,
but it seems better to delete the pointless equality methods which are
causing the problem.

4 years agocatacomb/__init__.py: Implement equality and hashing for `KeyData' objects.
Mark Wooding [Mon, 25 Nov 2019 12:07:16 +0000 (12:07 +0000)]
catacomb/__init__.py: Implement equality and hashing for `KeyData' objects.

Equality is determined by value, so don't use `KeyData' objects as
hashtable keys and then mutate them.

4 years agokey.c: Add hashing and comparison for `Key' objects.
Mark Wooding [Mon, 28 Oct 2019 10:49:55 +0000 (10:49 +0000)]
key.c: Add hashing and comparison for `Key' objects.

4 years agokey.c, catacomb/__init__.py: Split key file I/O into Python and C pieces.
Mark Wooding [Tue, 22 Oct 2019 11:35:40 +0000 (12:35 +0100)]
key.c, catacomb/__init__.py: Split key file I/O into Python and C pieces.

The new Catacomb `key_mergeline' and `key_extractline' functions were
specifically added to interface with language-specific I/O systems which
couldn't easily present as a `FILE *'.  Expose these new functions, and
implement the old file-oriented methods in terms of the new bindings.

4 years agokey.c: Make `None' be the default report-function designator.
Mark Wooding [Tue, 22 Oct 2019 11:34:48 +0000 (12:34 +0100)]
key.c: Make `None' be the default report-function designator.

Now we can pass it in from Python as an explicit argument.

4 years agobuffer.c: Publish the internal structure of buffer objects.
Mark Wooding [Mon, 21 Oct 2019 11:01:48 +0000 (12:01 +0100)]
buffer.c: Publish the internal structure of buffer objects.

4 years agobuffer.c: Add a lock count which pins a write-buffer's backing store.
Mark Wooding [Tue, 22 Oct 2019 16:14:46 +0000 (17:14 +0100)]
buffer.c: Add a lock count which pins a write-buffer's backing store.

Python 3's buffer protocol allows a long-term acquisition of a buffer's
backing-store address.  It would be Bad if one of our `WriteBuffer'
objects were to reallocate its backing store while something still
thought it had the old address, so keep track of how many times the
buffer has been referenced in this way.

Nothing actually makes use of this machinery yet, but its time will come.

4 years agobuffer.c: Funnel control through a common exit point.
Mark Wooding [Tue, 22 Oct 2019 10:29:09 +0000 (11:29 +0100)]
buffer.c: Funnel control through a common exit point.

I'm not going on a rampage to fix this throughout the code (yet?), but
while I'm here I might as well.

4 years agobuffer.c: Handle curve object as `PyObject *'.
Mark Wooding [Tue, 22 Oct 2019 10:27:49 +0000 (11:27 +0100)]
buffer.c: Handle curve object as `PyObject *'.

I've not checked, but I think this is the only place where a
pre- existing curve object was handled as `PyTypeObject *'.

4 years agobuffer.c: Delete pointless initialization.
Mark Wooding [Tue, 22 Oct 2019 10:27:12 +0000 (11:27 +0100)]
buffer.c: Delete pointless initialization.

The variable `cobj' will be set by `PyArg_ParseTuple' anyway.

4 years agopock: Use floor division on integers.
Mark Wooding [Tue, 22 Oct 2019 11:18:41 +0000 (12:18 +0100)]
pock: Use floor division on integers.

Python 3 wants to give me a float here, which is obviously wrong.

4 years agopock: Probe for the fixnum width by enquiring the largest fixnum value.
Mark Wooding [Tue, 22 Oct 2019 11:16:37 +0000 (12:16 +0100)]
pock: Probe for the fixnum width by enquiring the largest fixnum value.

Python 3 doesn't distinguish fixnums from bignums at the type level, so
this would run forever.  Instead, get (a guess at) the largest fixnum
value, and use that.

4 years agopock: Use `MPMul' so that we can avoid `reduce'.
Mark Wooding [Tue, 22 Oct 2019 11:14:10 +0000 (12:14 +0100)]
pock: Use `MPMul' so that we can avoid `reduce'.

For some reason, Python 3 hides `reduce' away in some library module.
Anyway, we have a very efficient mass-multiplier so we might as well use
it.

4 years agopwsafe: Trim an overly-long section boundary.
Mark Wooding [Mon, 21 Oct 2019 17:03:20 +0000 (18:03 +0100)]
pwsafe: Trim an overly-long section boundary.

4 years agocatacomb/pwsafe.py: Use `binascii' for Base64 conversion.
Mark Wooding [Mon, 21 Oct 2019 17:26:39 +0000 (18:26 +0100)]
catacomb/pwsafe.py: Use `binascii' for Base64 conversion.

The `.encode()' and `.decode()' string methods were apparently too
convenient, so Python 3 doesn't do Base64 conversion like this.
Instead, we have to use the steam-powered `binascii'.  And, to make
things even better, encoding produces `bytes' rather than `str' because,
err, it's not really text or something?  No idea.  Stoats.

4 years agocatacomb/pwsafe.py: Hack around the change in metaclass syntax.
Mark Wooding [Mon, 21 Oct 2019 17:17:21 +0000 (18:17 +0100)]
catacomb/pwsafe.py: Hack around the change in metaclass syntax.

Python 2 (all versions) want you to set `__metaclass__' within the class
body.  Python 3 ignores this, treating it like an ordinary class
attribute, and wants you to say `metaclass = ...' in the direct-
superclass list; but the latter is a syntax error to Python 2.

Instead, add a hack which constructs a anonymous class with the right
metaclass to use as a direct superclass.  It's awful, but it works.

4 years agocatacomb/pwsafe.py: Mark abstract storage-backend classes as lacking `NAME'.
Mark Wooding [Mon, 21 Oct 2019 17:20:42 +0000 (18:20 +0100)]
catacomb/pwsafe.py: Mark abstract storage-backend classes as lacking `NAME'.

Previously, they'd have a name of `None'.  This will make a later hack
work better.

4 years agocatacomb/pwsafe.py: Hack around the difference in octal literal syntax.
Mark Wooding [Mon, 21 Oct 2019 17:06:31 +0000 (18:06 +0100)]
catacomb/pwsafe.py: Hack around the difference in octal literal syntax.

Python 2.5 wants `0DIGITS', while Python 3 insists on `0oDIGITS'.  The
latter is prettier, but the difference is annoying.  We only need these
for Unix permissions, so just set up the ones we want in advance using
an explicit conversion from strings.

4 years agocatacomb/__init__.py: Implement `rich' comparisons on rationals.
Mark Wooding [Sun, 20 Oct 2019 01:29:36 +0000 (02:29 +0100)]
catacomb/__init__.py: Implement `rich' comparisons on rationals.

These are the only kind in Python 3, and they work everywhere.

4 years agocatacomb/__init__.py: Don't try to convert text strings to `ByteString'.
Mark Wooding [Mon, 21 Oct 2019 19:06:31 +0000 (20:06 +0100)]
catacomb/__init__.py: Don't try to convert text strings to `ByteString'.

This won't work in Python 3, so get used to the idea.

Fortunately, this only matters for the empty string, so use
`ByteString.zero(0)' instead.

4 years agocatacomb/__init__.py: Hack because Python 3 `hex' builtin works differently.
Mark Wooding [Mon, 21 Oct 2019 19:01:45 +0000 (20:01 +0100)]
catacomb/__init__.py: Hack because Python 3 `hex' builtin works differently.

In Python 3, the `hex' builtin function converts its operand to an
integer (using the new `__index__' special method) and then converts
that to hex, rather than calling a special method directly.  The new
`bytes' type offers a `hex' method for conversion to hex, so we do the
same here.

4 years agocatacomb/__init__.py: Hack because Python 3 won't destructure arguments.
Mark Wooding [Mon, 21 Oct 2019 17:25:28 +0000 (18:25 +0100)]
catacomb/__init__.py: Hack because Python 3 won't destructure arguments.

Why?!

4 years ago*.py: Hack around the mapping change from `itervalues' to just `values', etc.
Mark Wooding [Mon, 21 Oct 2019 17:13:57 +0000 (18:13 +0100)]
*.py: Hack around the mapping change from `itervalues' to just `values', etc.

This change is only about accessing existing mappings.  We'll have to
set up our own mappings the hard way.

4 years ago*.py: Add explicit conversions between binary and text strings.
Mark Wooding [Mon, 21 Oct 2019 17:04:58 +0000 (18:04 +0100)]
*.py: Add explicit conversions between binary and text strings.

This doesn't do anything right now, but it'll be an important hook for
porting to Python 3.

4 years ago*.py: Add parentheses around `print' arguments.
Mark Wooding [Mon, 21 Oct 2019 16:57:29 +0000 (17:57 +0100)]
*.py: Add parentheses around `print' arguments.

For `print >>FILE', just switch to `FILE.write(...)' instead, with an
explicit newline.

4 years ago*.py: Hack around Python exception-catching syntax mismatch.
Mark Wooding [Mon, 21 Oct 2019 16:53:43 +0000 (17:53 +0100)]
*.py: Hack around Python exception-catching syntax mismatch.

There's no common syntax to capture the current exception value between
Python 2.5 and 3: the former wants

try: ...
except EXCCLS, VAR: ...

while the latter wants

try: ...
except EXCCLS as VAR: ...

Give up and dig the exception value out of `sys.exc_info()' instead.

4 years ago*.py: Upgrade old-fashioned `raise' syntax.
Mark Wooding [Mon, 21 Oct 2019 16:47:30 +0000 (17:47 +0100)]
*.py: Upgrade old-fashioned `raise' syntax.

4 years agocatacomb/pwsafe.py: Add missing `.errno' projection.
Mark Wooding [Mon, 21 Oct 2019 16:52:35 +0000 (17:52 +0100)]
catacomb/pwsafe.py: Add missing `.errno' projection.

4 years agomp.c: Fix misformatting.
Mark Wooding [Mon, 21 Oct 2019 10:46:14 +0000 (11:46 +0100)]
mp.c: Fix misformatting.

4 years agoalgorithms.c: Delete a spurious blank line.
Mark Wooding [Tue, 19 Nov 2019 22:35:49 +0000 (22:35 +0000)]
algorithms.c: Delete a spurious blank line.

4 years agomp.c: Use newer names for the internal `long' representation parameters.
Mark Wooding [Mon, 21 Oct 2019 10:43:43 +0000 (11:43 +0100)]
mp.c: Use newer names for the internal `long' representation parameters.

In Python 2.6, the `SHIFT' and `MASK' constants grew `PyLong_' prefixes,
but the old names were retained for compatibility; in Python 3, the old
names are gone.  Use the new names, defining them as necessary.

4 years agomp.c: Assert about the Python limb size statically.
Mark Wooding [Mon, 21 Oct 2019 10:40:39 +0000 (11:40 +0100)]
mp.c: Assert about the Python limb size statically.

Python extensions build with assertions off, so this was much less
helpful than it looked.

4 years ago*.c: Separate string function calls according to text/binary usage.
Mark Wooding [Mon, 21 Oct 2019 01:38:59 +0000 (02:38 +0100)]
*.c: Separate string function calls according to text/binary usage.

Don't use the `PyString_...' functions directly.  Instead, define two
collections of macros `TEXT_...' and `BIN_...', which can be used on
text and binary strings respectively.  The two sets are slightly
different, because they've been designed to adapt to Python 3's
different interfaces.

There are no `PyString...' calls remaining outside of these macros.

4 years ago*.c: Introduce a new input conversion for binary strings.
Mark Wooding [Mon, 21 Oct 2019 00:43:54 +0000 (01:43 +0100)]
*.c: Introduce a new input conversion for binary strings.

One of the major differences in Python 3 is that it firmly distinguishes
between binary and text strings: the former consist of small integers,
while the latter consist of Unicode scalars.  The Python 3 `s#'
conversion's main purpose is to accept text strings, and though it will
also accept binary strings it's not really ideal for the purpose.
Python 3 introduces a new conversion `y#' specifically for binary
strings, though this isn't quite what we want because, for some reason,
it /doesn't/ work with bufferish objects which require explicit release.

The best answer seems to be to introduce our own custom conversion for
binary strings, so we do this here, replacing all of the binary-input
argument conversions.  While we're at it, replace all of the by-steam
argument conversions using `PyObject_AsReadBuffer' too.

4 years ago*.c: Use the new `Py_hash_t' type.
Mark Wooding [Sun, 20 Oct 2019 23:33:22 +0000 (00:33 +0100)]
*.c: Use the new `Py_hash_t' type.

4 years ago*.c: Use `PyVarObject_HEAD_INIT' to initialize type object headers.
Mark Wooding [Sun, 20 Oct 2019 23:14:01 +0000 (00:14 +0100)]
*.c: Use `PyVarObject_HEAD_INIT' to initialize type object headers.

Define a suitable version if the Python headers don't do it for us.

4 years ago*.c: Use the new `Py_TYPE' and `Py_SIZE' macros; define them if necessary.
Mark Wooding [Sun, 20 Oct 2019 23:19:44 +0000 (00:19 +0100)]
*.c: Use the new `Py_TYPE' and `Py_SIZE' macros; define them if necessary.

Python 2.6 introduced macros to access object header fields, which is
handy because Python 3 changes the way object headers are
structured.  (I think the binary layout is unchanged, but the C-level
structuring is definitely different.)

This is the start of Python 3 porting work.

4 years ago*.c: Publish algorithm and crypto-group tables as `mapping' objects.
Mark Wooding [Wed, 13 Nov 2019 00:59:07 +0000 (00:59 +0000)]
*.c: Publish algorithm and crypto-group tables as `mapping' objects.

Previously, the algorithm tables were published as plain Python
dictionaries, and the group tables were published in a complicated way
through a read-only mapping object implemented in Python.

With the richer Python 3 mapping protocol, maintaining a separate custom
mapping object in Python seems much less attractive.  Also, the group
tables were read-only while the algorithm tables were mutable, which was
a rather unfortunate inconsistency.

Fix this mess by implementing a new simple mapping object, based on
Pyke's generic mapping machinery, and populating instances of it with
the necessary data.

4 years agopyke/mapping.c, key.c: Make the mapping code more intrusive and complete.
Mark Wooding [Mon, 18 Nov 2019 10:41:12 +0000 (10:41 +0000)]
pyke/mapping.c, key.c: Make the mapping code more intrusive and complete.

Previously, mapping classes would implement a minimum of Python-level
operations, such as iterating over keys and mapping lookups, and the
generic mapping code would just synthesize missing operations in terms
of these.

Instead, introduce a mapping-specific vtable pointer in object
structures which describes the mapping operations in a form similar to
mLib's `sym' interface, which is rather more efficient in terms of hash
probes, and have the generic mapping code synthesize all of the Python-
level operations from these.

Needless to say, this is very messy.  Sorry.

4 years agopyke/mapping.c: Introduce macro for unconstifying common keyword list.
Mark Wooding [Sun, 20 Oct 2019 21:57:54 +0000 (22:57 +0100)]
pyke/mapping.c: Introduce macro for unconstifying common keyword list.

4 years agopyke/pyke.c (newtype): Explicitly clear `ht_slots'.
Mark Wooding [Tue, 22 Oct 2019 17:53:50 +0000 (18:53 +0100)]
pyke/pyke.c (newtype): Explicitly clear `ht_slots'.

This makes things go very badly wrong in Python 3.  I'm not sure why it
doesn't go wrong in Python 2...

4 years ago*.c: Split the constant definitions into the various submodules.
Mark Wooding [Sun, 20 Oct 2019 22:18:09 +0000 (23:18 +0100)]
*.c: Split the constant definitions into the various submodules.

4 years agopyke/pyke.[ch]: Make type skeleton structures be read-only.
Mark Wooding [Sun, 20 Oct 2019 20:18:08 +0000 (21:18 +0100)]
pyke/pyke.[ch]: Make type skeleton structures be read-only.

We couldn't do this before because `INITTYPE_META' would write the
direct superclass to the `tp_base' slot in the skeleton before calling
`inittype'.  To make this work, then, we adjust `inittype' to take the
direct superclass as an extra argument and plug it into the newly
created heap-type; and `INITTYPE_META' needs adjusting to pass this
argument rather than trying to write to the skeleton directly.

Of course, then we need to actually mark the type skeletons as `const'.

4 years agoalgorithms.c, ec.c, field.c: Replace properties by member access.
Mark Wooding [Tue, 19 Nov 2019 08:29:49 +0000 (08:29 +0000)]
algorithms.c, ec.c, field.c: Replace properties by member access.

4 years agoalgorithms.c: Fix longstanding ugly hack with new `MEMRNM' macro.
Mark Wooding [Sun, 20 Oct 2019 15:15:39 +0000 (16:15 +0100)]
algorithms.c: Fix longstanding ugly hack with new `MEMRNM' macro.

4 years agopyke/pyke.h: Add a `MEMBER' variant with explicit member name.
Mark Wooding [Sun, 20 Oct 2019 11:43:40 +0000 (12:43 +0100)]
pyke/pyke.h: Add a `MEMBER' variant with explicit member name.

4 years ago*.c: Use Python `METH_NOARGS' methods where applicable.
Mark Wooding [Sun, 20 Oct 2019 20:00:18 +0000 (21:00 +0100)]
*.c: Use Python `METH_NOARGS' methods where applicable.

Rather than rolling our own.  We need a little extra machinery to insert
the table entries, especially for the generic mapping support, but this
still saves 95 lines of code.

The `CONVERT_CAREFULLY' macro recently added will check that we haven't
messed things up too badly.

4 years ago*.c: Use Python's facilities for defining class and static methods.
Mark Wooding [Tue, 19 Nov 2019 00:19:38 +0000 (00:19 +0000)]
*.c: Use Python's facilities for defining class and static methods.

These weren't available when this library was first written, but the
workaround is rather ugly and no longer necessary.

4 years ago*.c: Rearrange and reformat the class methods.
Mark Wooding [Mon, 18 Nov 2019 19:53:29 +0000 (19:53 +0000)]
*.c: Rearrange and reformat the class methods.

This is preparation for using Python's built-in facilities for marking
static and and class methods (which weren't around when this project
started out, which is why things are the rather strange way that they
are).

4 years ago*.c: Make all of the type-definition tables read-only.
Mark Wooding [Sun, 20 Oct 2019 21:46:05 +0000 (22:46 +0100)]
*.c: Make all of the type-definition tables read-only.

The hard part is a new collection of macros which strip the `const'
qualifier from the actual table definitions, after checking that they
actually have the correct type.

And then there's the slog of actually changing all of the definitions
and using the new macros.

4 years agocatacomb-python.h (GEN): Remove `const' from input table type carefully.
Mark Wooding [Mon, 21 Oct 2019 12:07:43 +0000 (13:07 +0100)]
catacomb-python.h (GEN): Remove `const' from input table type carefully.

We have this new machinery, so we should use it.

4 years agopyke/pyke.c: Check conversions hidden inside `KWLIST' and `KWMETH'.
Mark Wooding [Sun, 20 Oct 2019 18:24:37 +0000 (19:24 +0100)]
pyke/pyke.c: Check conversions hidden inside `KWLIST' and `KWMETH'.

Introduce a new macro `CONVERT_CAREFULLY' which checks (via a rather
sleazy hack) that its operand is of the expected type before converting
it to some other type.  This is protected by an `#ifdef' guard because
I'm thinking about adding it to a version of mLib, but I want to keep
the Pyke core independent of mLib.

Use this new macro to build better versions of `KWLIST' and `KWMETH',
which have casts hidden inside them, so that we can now be certain that
the method table entries match up with the functions.  (Spoiler: they
didn't, quite, and commit 19bff42f99d41cd9b8953ff61edfe35380b54d88,
backported onto the 1.1.x branch, fixes the bug that was found by this
change.)

4 years agopyke/pyke.h, key.c: Rename `INDEXERR' to `MAPERR'.
Mark Wooding [Sun, 20 Oct 2019 18:17:41 +0000 (19:17 +0100)]
pyke/pyke.h, key.c: Rename `INDEXERR' to `MAPERR'.

The name `INDEXERR' was pointlessly confusing, given the existing
`IXERR' which actually raises Python's `IndexError'.  It's important,
though, that `KEYERR' be reserved for Catacomb's `KeyError' exception,
which really is an error about (cryptographic) keys.  So `MAPERR' it is.

4 years agoSplice 'pyke/' prehistory from commit 'c1756f78cbf3007438b3eb2fbfbd632d070be6ca'
Mark Wooding [Fri, 10 Apr 2020 22:27:20 +0000 (23:27 +0100)]
Splice 'pyke/' prehistory from commit 'c1756f78cbf3007438b3eb2fbfbd632d070be6ca'

git-subtree-dir: pyke
git-subtree-mainline: 10e6f88a407ce04807b6ad76a81be553f8d7abaa
git-subtree-split: c1756f78cbf3007438b3eb2fbfbd632d070be6ca

(This commit was built by hand to splice the synthetic prehistory onto
the `pyke/' subtree history.)

* pyke-prehistory.head: (83 commits)
  pyke/, ...: Extract utilities into a sort-of reusable library.
  (Pruned history of `catacomb-python' begins here.)
  catacomb-python.h: Delete a stray trailing `\'.
  catacomb-python.h: Add a macro to declare module init functions.
  util.c: Replace mLib `DISCARD' with a plain `(void)' cast.
  util.c: Rewrite `addmethods' to remove dependency on <mLib/darray.h>.
  catacomb.c: Use a less awful version comparison.
  catacomb.c, util.c: Export `modname' and set it in main entry point.
  *.c: Use `Py_XDECREF' where applicable.
  *.c: Reformat docstrings.
  catacomb-python.h: Give up on Python versions prior to 2.5.
  *.c: Use Python macros rather than functions where possible.
  catacomb-python.h, *.c: Move definitions back into implementation files.
  Merge branch '1.3.x'
  Merge branch '1.2.x' into 1.3.x
  *.c: Be more careful about `PySequence_Size'.
  catacomb-python.h: Add a macro for raising `OverflowError'.
  util.c (mkexc): Populate dictionary before constructing exception class.
  catacomb.c, util.c: Publish negative constants correctly.
  field.c: Delete the completely unused `getfe' function.
  catacomb.c: Publish `RAND_IBITS' constant.
  ...

4 years agopyke/, ...: Extract utilities into a sort-of reusable library.
Mark Wooding [Sun, 20 Oct 2019 16:51:13 +0000 (17:51 +0100)]
pyke/, ...: Extract utilities into a sort-of reusable library.

This commit changes no code, but it moves a lot of it about.  Tidying up
will come later.

4 years ago(Pruned history of `catacomb-python' begins here.)
Mark Wooding [Fri, 10 Apr 2020 22:02:06 +0000 (23:02 +0100)]
(Pruned history of `catacomb-python' begins here.)

The code which is now the `Pyke' library used to be part of the Catacomb
Python bindings.  Beginning here is a pruned version of that history,
limited to the files which actually contributed to the `Pyke' codebase.
It was constructed from the original history, starting at revision
10e6f88a407ce04807b6ad76a81be553f8d7abaa, by running

git filter-branch --prune-empty --index-filter '
git ls-files -s | {
  mode=old
  keep=
  while read m h s f; do
    case $mode,$f in
      old,util.c | old,catacomb.c | old,catacomb-python.h)
keep="$keep $m,$h,$f" ;;
      old,pyke/*)
keep="$m,$h,${f#*/}" mode=new ;;
      new,pyke/*)
keep="$keep $m,$h,${f#*/}" ;;
    esac
    git update-index --force-remove $f
  done
  for i in $keep; do
    git update-index --add --cacheinfo $i
  done
}
' pyke-prehistory.head -- pyke/ util.c catacomb.c catacomb-python.h

and the tucking this empty commit beneath the head which actually
introduces the `pyke/' directory.  (This subterfuge is, unfortunately,
necessary for `git subtree' to construct the right history.)

4 years agopyke/, ...: Extract utilities into a sort-of reusable library.
Mark Wooding [Sun, 20 Oct 2019 16:51:13 +0000 (17:51 +0100)]
pyke/, ...: Extract utilities into a sort-of reusable library.

This commit changes no code, but it moves a lot of it about.  Tidying up
will come later.

4 years agocatacomb-python.h: Delete a stray trailing `\'.
Mark Wooding [Tue, 22 Oct 2019 18:22:07 +0000 (19:22 +0100)]
catacomb-python.h: Delete a stray trailing `\'.

This doesn't actually hurt anything because of the following blank line,
but, umm, ...

4 years agocatacomb-python.h: Delete a stray trailing `\'.
Mark Wooding [Tue, 22 Oct 2019 18:22:07 +0000 (19:22 +0100)]
catacomb-python.h: Delete a stray trailing `\'.

This doesn't actually hurt anything because of the following blank line,
but, umm, ...

4 years agocatacomb-python.h: Add a macro to declare module init functions.
Mark Wooding [Sun, 20 Oct 2019 17:18:05 +0000 (18:18 +0100)]
catacomb-python.h: Add a macro to declare module init functions.

4 years agocatacomb-python.h: Add a macro to declare module init functions.
Mark Wooding [Sun, 20 Oct 2019 17:18:05 +0000 (18:18 +0100)]
catacomb-python.h: Add a macro to declare module init functions.

4 years agoutil.c: Replace mLib `DISCARD' with a plain `(void)' cast.
Mark Wooding [Sun, 20 Oct 2019 17:56:07 +0000 (18:56 +0100)]
util.c: Replace mLib `DISCARD' with a plain `(void)' cast.

4 years agoutil.c: Replace mLib `DISCARD' with a plain `(void)' cast.
Mark Wooding [Sun, 20 Oct 2019 17:56:07 +0000 (18:56 +0100)]
util.c: Replace mLib `DISCARD' with a plain `(void)' cast.

4 years agoutil.c: Rewrite `addmethods' to remove dependency on <mLib/darray.h>.
Mark Wooding [Sun, 20 Oct 2019 17:03:35 +0000 (18:03 +0100)]
util.c: Rewrite `addmethods' to remove dependency on <mLib/darray.h>.

4 years agoutil.c: Rewrite `addmethods' to remove dependency on <mLib/darray.h>.
Mark Wooding [Sun, 20 Oct 2019 17:03:35 +0000 (18:03 +0100)]
util.c: Rewrite `addmethods' to remove dependency on <mLib/darray.h>.

4 years agocatacomb.c: Use a less awful version comparison.
Mark Wooding [Sun, 20 Oct 2019 20:03:38 +0000 (21:03 +0100)]
catacomb.c: Use a less awful version comparison.

4 years agocatacomb.c: Use a less awful version comparison.
Mark Wooding [Sun, 20 Oct 2019 20:03:38 +0000 (21:03 +0100)]
catacomb.c: Use a less awful version comparison.

4 years agocatacomb.c, util.c: Export `modname' and set it in main entry point.
Mark Wooding [Sun, 20 Oct 2019 16:57:11 +0000 (17:57 +0100)]
catacomb.c, util.c: Export `modname' and set it in main entry point.

This is preliminary work in splitting out the Catacomb-independent
utilities so that they can be used by other projects.

4 years agocatacomb.c, util.c: Export `modname' and set it in main entry point.
Mark Wooding [Sun, 20 Oct 2019 16:57:11 +0000 (17:57 +0100)]
catacomb.c, util.c: Export `modname' and set it in main entry point.

This is preliminary work in splitting out the Catacomb-independent
utilities so that they can be used by other projects.

4 years ago*.c: Use `Py_XDECREF' where applicable.
Mark Wooding [Sat, 19 Oct 2019 13:31:40 +0000 (14:31 +0100)]
*.c: Use `Py_XDECREF' where applicable.

The Python 2.5 version of `Py_DECREF' isn't properly braced, so `if (x)
Py_DECREF(x);' provokes a dangling-`else' warning from the compiler.

4 years ago*.c: Use `Py_XDECREF' where applicable.
Mark Wooding [Sat, 19 Oct 2019 13:31:40 +0000 (14:31 +0100)]
*.c: Use `Py_XDECREF' where applicable.

The Python 2.5 version of `Py_DECREF' isn't properly braced, so `if (x)
Py_DECREF(x);' provokes a dangling-`else' warning from the compiler.

4 years ago*.c: Reformat docstrings.
Mark Wooding [Fri, 22 Nov 2019 20:30:31 +0000 (20:30 +0000)]
*.c: Reformat docstrings.

No functional changes here: just changing how the strings are
represented in the source.

4 years ago*.c: Reformat docstrings.
Mark Wooding [Fri, 22 Nov 2019 20:30:31 +0000 (20:30 +0000)]
*.c: Reformat docstrings.

No functional changes here: just changing how the strings are
represented in the source.

4 years agocatacomb-python.h: Give up on Python versions prior to 2.5.
Mark Wooding [Fri, 22 Nov 2019 22:43:02 +0000 (22:43 +0000)]
catacomb-python.h: Give up on Python versions prior to 2.5.

4 years agomp.c: Improve docstring.
Mark Wooding [Fri, 22 Nov 2019 20:22:45 +0000 (20:22 +0000)]
mp.c: Improve docstring.

4 years ago*.c: Use Python macros rather than functions where possible.
Mark Wooding [Sun, 24 Nov 2019 12:35:45 +0000 (12:35 +0000)]
*.c: Use Python macros rather than functions where possible.

4 years agopgen.c: Write a `pgen' status code as `PGST' consistently in docstrings.
Mark Wooding [Fri, 22 Nov 2019 18:34:56 +0000 (18:34 +0000)]
pgen.c: Write a `pgen' status code as `PGST' consistently in docstrings.

4 years agocatacomb-python.h, *.c: Move definitions back into implementation files.
Mark Wooding [Sun, 20 Oct 2019 14:56:54 +0000 (15:56 +0100)]
catacomb-python.h, *.c: Move definitions back into implementation files.

The header was chock full of implementation details which weren't
needed by anything.  Move things which aren't needed further afield back
into their home files, and delete things like conversion functions which
aren't actually being used.

Also, some light reordering of the things that are left.

(This patch turns out to be remarkably commutative through the enormous
pile of changes coming up.)

4 years agokey.c: Delete the `barf' function.
Mark Wooding [Sun, 20 Oct 2019 21:53:24 +0000 (22:53 +0100)]
key.c: Delete the `barf' function.

What even?

I think this must have been ancient debugging code, but it should never
have seen release.

4 years agobytestring.c: Delete unused buffer methods on `C.ByteString'.
Mark Wooding [Sun, 20 Oct 2019 21:07:40 +0000 (22:07 +0100)]
bytestring.c: Delete unused buffer methods on `C.ByteString'.

I think this was needed in some ancient Python in order to inherit the
buffer methods from `str', but Python 2.5 certainly works fine without.

4 years agobuffer.c: Give `BufferError' a text message.
Mark Wooding [Sun, 20 Oct 2019 20:08:32 +0000 (21:08 +0100)]
buffer.c: Give `BufferError' a text message.

4 years agocatacomb-python.h: Give up on Python versions prior to 2.5.
Mark Wooding [Fri, 22 Nov 2019 22:43:02 +0000 (22:43 +0000)]
catacomb-python.h: Give up on Python versions prior to 2.5.

4 years agofield.c: Push field-polynomial property into binary-field base class.
Mark Wooding [Wed, 23 Oct 2019 09:41:34 +0000 (10:41 +0100)]
field.c: Push field-polynomial property into binary-field base class.

Using the same code as for prime fields was a mistake, but sharing the
same code between polynomial- and normal-basis fields works just fine;
but there's no point in duplicating the property-table entry when
there's a perfectly good superclass to hang the property off -- and it
already has a property table!

4 years ago*.c: Use Python macros rather than functions where possible.
Mark Wooding [Sun, 24 Nov 2019 12:35:45 +0000 (12:35 +0000)]
*.c: Use Python macros rather than functions where possible.

4 years agocatacomb-python.h, *.c: Move definitions back into implementation files.
Mark Wooding [Sun, 20 Oct 2019 14:56:54 +0000 (15:56 +0100)]
catacomb-python.h, *.c: Move definitions back into implementation files.

The header was chock full of implementation details which weren't
needed by anything.  Move things which aren't needed further afield back
into their home files, and delete things like conversion functions which
aren't actually being used.

Also, some light reordering of the things that are left.

(This patch turns out to be remarkably commutative through the enormous
pile of changes coming up.)

4 years ago.gitignore: Reorder the patterns and make them more specific.
Mark Wooding [Sun, 24 Nov 2019 12:08:34 +0000 (12:08 +0000)]
.gitignore: Reorder the patterns and make them more specific.

Place recursively applicable patterns at the top (just `*.pyc' for
now).  Prefix others with `/' to keep them local.  Suffix directory
names with `/' for clarity's sake.

4 years ago.gitignore: Delete some obsolete things.
Mark Wooding [Sun, 24 Nov 2019 11:35:32 +0000 (11:35 +0000)]
.gitignore: Delete some obsolete things.

I think `dist' and `deb-build' were from old private workflow that
shouldn't have leaked into the master repository.  I have a vague
recollection that `build-stamp' was from CDBS, which we no longer use.
I honestly have no idea what `py' and `test.py' were, but they've been
there from the beginning.

4 years agocatacomb/.gitignore: Delete obsolete file.
Mark Wooding [Sun, 24 Nov 2019 12:05:48 +0000 (12:05 +0000)]
catacomb/.gitignore: Delete obsolete file.

I think this was only created because Subversion ignore patterns didn't
affect subdirectories.

4 years ago.gdbinit: Delete this obsolete file. master
Mark Wooding [Tue, 7 Apr 2020 23:53:56 +0000 (00:53 +0100)]
.gdbinit: Delete this obsolete file.

It's set up for my particular development environment, using a Python
version that's not even supported any more.  And project-specific
`.gdbinit' files really don't work very well as things to distribute,
for vaguely obvious security reasons.

4 years agoMerge branch '1.3.x' into HEAD
Mark Wooding [Tue, 7 Apr 2020 23:59:00 +0000 (00:59 +0100)]
Merge branch '1.3.x' into HEAD

* 1.3.x:
  t/t-rand.py: Set the correct refernce seed value for `dsarand'.
  rand.c: More `Py_ssize_t' fixes.

4 years agot/t-rand.py: Set the correct refernce seed value for `dsarand'. 1.3.x
Mark Wooding [Tue, 7 Apr 2020 23:40:52 +0000 (00:40 +0100)]
t/t-rand.py: Set the correct refernce seed value for `dsarand'.

The number of steps taken is different on 32- and 64-bit platforms, with
no especially easy way to calculate it in a principled way.  Just use a
stupid case analysis to get the right answer.

4 years agoMerge branch '1.2.x' into 1.3.x
Mark Wooding [Tue, 7 Apr 2020 23:56:01 +0000 (00:56 +0100)]
Merge branch '1.2.x' into 1.3.x

* 1.2.x:
  rand.c: More `Py_ssize_t' fixes.

4 years agorand.c: More `Py_ssize_t' fixes.
Mark Wooding [Tue, 7 Apr 2020 23:19:14 +0000 (00:19 +0100)]
rand.c: More `Py_ssize_t' fixes.