Make the code -Wconversion ready: explicit casts only

This patch, while adding lots of extra boilerplate, converts all code
to explicit casts only; it now builds with `-Wconversion` cleanly,
although that's a bit too strong to turn on by default (for now).

Fix a few int/size_t conversion issues

From https://bugzilla.redhat.com/show_bug.cgi?id=1127310: the
'#'-variats of parsing (s#, et#, etc.) take a size_t if
PY_SSIZE_T_CLEAN is defined (which the code does). On some
architectures (little-endian only?) this doesn't seem to be a problem,
but on PPC this is a failure in unit tests (at least yay).

The patch changes the type of bufsize to Py_ssize_t, and does an
explicit cast to size_t with underflow check in order to make the code
safer (assuming this is the right thing to do). This should result in
no unexpected conversion issues.

Improve error checking in get_all()

After getting a too big value via ERANGE, making the call with the
proper length could still fail due to other issues, and this check is
missing. A negative value here would be changed into something weird
in the realloc call (which takes size_t), so the error reporting would
be bogus.

Style change: name all jump labels uniformly

Fix memory leak on get_all OutOfMemory handling path

It's unlikely that the situation is recoverable when failing to
allocate memory, but the current code is clearly buggy: PyMem_Realloc
doesn't clobber the existing buffer on failure, so not deallocating it
will result in a leak.

Update my email address

Bump upper python version test in Makefile

Bump version for 0.5.4 release

Remove all .so files on make clean

Update copyright year

Sigh, boring.

Fix memory leaks in get() failure handling path

The freeing memory labels were used wrongly; easily reproducible by
simulating failures in a loop.

Update changelog and bump version for 0.5.3

Add helpers to test suite for easier checking

Since the _ignore/_ignore_tuples methods are always called with
assertEqual, let's add two wrappers to simplify the code a bit.

Make the environment variable TEST_DIR match the python var

Having things inconsistent just leads to confusion.

Merge pull request #3 from jborg/surrogateescape

Use "surrogateescape" error handler when encoding unicode paths.

Merge pull request #4 from edelkind/master

Better handling of files without xattrs

Merge pull request #5 from quantheory/master

Add workaround for tests on SELinux systems.

Fix tests when tmp files have attributes.

SELinux systems may have the security.selinux attribute defined for
temporary files upon creation. This can cause problems for the tests,
which assumed that new files have no attributes. Similar problems can
arise in other cases where temporary files created by python start out
with attributes.

To work around this, a comma-separated list of attributes to ignore can
be specified in the environment variable "TEST_IGNORE_XATTRS". Any
attributes specified in this way will be filtered out by tests.

Better handling of files without xattrs

Return an empty list immediately if a file has no xattrs (as most would
not on most systems).  The number of system calls for files with no
extended attributes is reduced by (at least) half.

Use "surrogateescape" error handler when encoding unicode paths.

On Python 3 functions like os.listdir() always returns unicode
paths. "invalid" byte paths are decoded using the "surrogateescape" error
handler which must also be used when decoding.
See os.fsencodefs() and os.fsdecode().

Update docs for the 0.5.2 release

Of course I forgot this, and as github has also deprecated downloads,
we should switch to something else…

Bump version and update NEWS for the 0.5.2 release

All tests pass, let's release!

Fix string/unicode buffer size parsing

It seems that CPython allows passing "wrong" pointers for et#
(Py_ssize_t instead of int), but PyPy has problems with this, and
wrongly casts the pointers, leading to unsigned/signed conversion
failures.

This bug was introduced in commits 433c5852 and 8aa6fd81; I'm no
longer sure why bufsize was changed from int to Py_ssize_t as well,
but passing an int value to a C function expecting an ssize_t should
be safe as the compiler should do the required conversions behind the
scenes (as long as it has correct information about the types).

Closes gh-1.

Enable testing with python 3.3 and pypy as well

The pypy test is not versioned, since Debian only provides a single
pypy binary, so I don't have an environment with multiple pypy
installations.

Enable compiler warnings

`-Wall -Werror` should make compiler warnings much more obvious,
hopefully catching more latent bugs.

Make setup.py behave nicely without setuptools

Currently, it would fail hard, whereas we can at least build the
extension (even though the testsuite will not work) when setuptools is
missing.

Python 2.5 can't parse byte literals, so use bytes() instead.

Fix title in index page

Clarify the namespace type under Py3

Update NEWS file for the 0.5.1 release

Complete the change of namespace handling

This disables None as a valid namespace under Python 2.x too; now only
an empty string is accepted.

The patch also documents the change.

Fix bug in get_all for empty namespace handling

Also add an unit test for it.

More setup.py updates

Add a download_url for pypi

Bump version number for new release

Also switch the homepage URL.

Update gitignore file

Makefile improvements

… and remove old run-tests file, obsoleted by the Makefile.

Update manifest file

Update README to include the old website info

Convert NEWS file

This allows it to be included in the doc nicely.

Switch documentation style to sphinx

Still work needed to transition this completely, but looks good so far.

Rename convertObj → convert_obj

Makes the style more consistent with the rest of the file.

Fix two missing close() class in tests

Eliminates a warning during testing.

Rework parsing of the namespace argument

In Python 3, we cannot accept (easily) bytes or None, so we have to
change how we accept the namespace argument.

Previously, if the argument was not passed or it was None, it was
considered missing. Since the None option is no longer possible, we
change it so that if not passed or passed as an empty string, it is
considered missing. This changes the behaviour somewhat, but I hope
that empty namespaces are not used (that's what I understand from
reading various pages on the internet).

Remove use of deprecated fail* test functions

Rework init sequence

This makes it a bit more clean, and thus removes the last refcount
issue; however, the code itself is much uglier :(

A few docstring fixes

Fix a (harmless) integer mismatch

This is not critical, but let's not over-optimise and request a byte
when we store it anyway in an int…

Check merge_ns return value in all places

This eliminates another potential round of errors. Also add cpychecker
attribute for even less noise.

Rework merge_ns signature

This is also in order to clarify what is an "error" return for
cpychecker.

Note that most calls to this function do not have yet error checking,
will be fixed in separate patch.

Fix a case of missing exception on return NULL

Now that the noise is gone, this is an actual error.

Rename a variable for more consistency

At least, more consistency in the output of git grep convertObj :)

Mark convertObj as setting the exception on errors

This eliminates lots of clutter from the cpychecker output, yay!

Switch converObj to negative error return

This will allow cpychecker to know that this function has set the
exception already, eliminating false positives.

Fix some const char* pointers

This reduces the warnings, and it's a good thing to do.

Switch to setuptools

For proper testsuite support.

Fix small typo

Fix some potential issues if PyBytes_FromString fails

This is unlikely, but if it happens it's ugly.

Fix bugs reported by cpychecker

Thanks to Dave Malcolm's cpychecker tool, this patch fixes a number of
serious issues. All issues that were not deemed false-positives were
fixed; some other issues in the same category that were found only by
a high number of refcount checks are also fixed (I should
split/simplify some parts of the code…).

Increase version to 0.5.0

Improve the unittest runner

This uses only existing python interpreters and does a cleanup before
building the unittest, to help with stale objects.

Fix a size_t/ssize_t mismatch

ACL library functions return a ssize_t, so all variables that hold
return values from such return types must be ssize_t, otherwise
comparisons with -1 will fail.

Small style fixes

Add python 3 support

This big patch does a few things (which are hard to split):
  - uniformize all function w.r.t. the error exit path (all functions
    now use only one exit point)
  - move from 's' to 'et' format in ParseArgs so that we can accept both
    binary and text, in both 2.x and 3.0
  - update the unittest for py3k

Initialize local variables

This patch prepares for the "et" format conversion by initializing the
local variables; it also adds a new "res" object for returns and adds a
few style fixes.

Add a new field to the target_t structure

The patch extends the target_t structure with a new PyObject field and
adds a cleanup function for it. This field is needed for properly
handling unicode input arguments with Py3K.

Make the module setup py3k-compatible

This patch converts the initialization of the module to work with both
2.x and 3.0.

Add script to run tests

Since we don't have setuptools for now, we add a script to run tests and
modify the test script to work without setuptools.

Switch back to distutils

Python 3.0 doesn't yet have setuptools, so temporarily we switch back to
distutils.

Add license information to source file

Increase version to 0.4.0

Update MANIFEST.in file for the release

Some final docstring touches before release

Update license to LGPLv2.1 or later

unittests: enhance symlink tests

The patch adds support for testing the behaviour of non-dangling
symlinks; we test that indeed attributes set on the target file are not
visible on the symlink when using nofollow.

Some more docstring updates

Docstring updates

More epydoc features are now used for prettier output.

More PEP 353 compliancy (new-style functions)

Reorder some declarations

It's better to have the 'system' defs before our user defs.

Update .gitignore

Merge branch 'fixes' into new_style

Fixed conflicts in:

xattr.c

Implement changes needed for PEP 353 compliance

It seems the coded wrongly mixed ssize_t and ints in a couple of places,
plus not properly using Py_ssize_t as needed. Fix these and ensure
backwards compat with python 2.4.

unittests: add many checks for namespace ops

The patch adds in many places checks for result correctness when using
namespaces, and for the equality of non-ns versus ns usage.

Change the get_all function when using namespaces

This patch changes the returned names from get_all() so that if a
namespace was passed, it and the prefix will be stripped from the names.

Also the docstring for list() was changed to have the same wording as
this one with regard to the names returned.

Change the match_ns function syntax

This patch changes the order of the two arguments and makes the function
return either NULL (failure) or a pointer to the (possibly shortened)
name, so that getting the short name is easier.

Docstring updates

Add new-style list function

unittests: allow changing the test dir

Add new-style remove function

Add the new-style get function

Correct the signature of the generic tgt methods

Add new-style set() function

This patch adds a new style set function and a generic merge_ns function
for combining a namespace string and an attribute name into the final
name.

The patch also duplicates all test methods for dual testing of new-style
and deprecated functions.

Some small fixes in get_all

Add unittests for the get_all function

Merge branch 'fixes' into multiget

Simplify the return None cases

Merge branch 'fixes' into multiget

Rename dolink to nofollow

This name makes more sense.

Merge branch 'fixes' into multiget

Convert to object protocol

This patch changes the hardcoded if-then-else constructs for selecting
the type of function (file-descriptor, symlink, normal path) into a
generic framework.