| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
However: "cgensupport.h" is still present... otherwise I get
maaaany type errors... not sure if this needs more attention.
|
|
|
|
|
|
|
|
|
|
| |
non-checked error return values, and where appropriate,
PyArg_ParseTuple() style argument parsing.
I also changed some function names and converted all malloc/free calls
to PyMem_NEW/PyMem_DEL.
Some stylistic changes and formatting standardization.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Where optional arguments were being used, converted to
PyArg_ParseTuple() style instead of nested PyArg_Parse() style.
- Check for and handle many potential error conditions that were never
being tested.
- internal reg_* functions renamed to regobj_* (makes it easier to
figure out which are global regex functions and which are for regex
objects).
- reg_group (now regobj_group) was quite extensively reworked. it no
longer recurses to do its job (by factoring core functionality into
a separate function that knows about string and integer indexes).
- some minor formatting fixes.
- regex_set_syntax() now invalidates the cache. Without this change
(in the example below), the second search would produce different
output depending on whether the first search were performed or not
(since performing the first search would cache the compiled object
with RE_SYNTAX_EMACS, causing the second test to unexpectedly fail).
regex.search('(a+)|(b+)', 'cdb')
prev = regex.set_syntax(RE_SYNTAX_AWK)
regex.search('(a+)|(b+)', 'cdb')
|
| |
|
|
|
|
| |
2. Reworked error checking in initposix() and initnt().
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
substituted the appropriate PyArg_Parse calls. Retested. All appears well.
|
|
|
|
| |
proper functions as defined in rename2.h. Thanks Barry!
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Watch it, Barry! :-)
|
| |
|
|
|
|
|
| |
style corrections, I presume), found by Jack. Added warning that this
has not been tested (Jack could only compile and link it).
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
This is needed because if a configure option has " as its value,
it will be rendered as {"}; after stripping one level of quoting it's
just ", on which splitlist will barf.
|
| |
|
| |
|
| |
|
|
|
|
| |
checking nits.
|
|
|
|
|
|
|
| |
SIGCPU and SIGFSZ but we're (Jeremy and I) are actually unsure whether
these were typos or if there are systems that use these alternate
names. We've checked Solaris, SunOS, and IRIX; they contain only the
SIGX* names.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Alas, I don't have an Indigo, so I could not even compile this.
|
|
|
|
|
| |
out manually, and it does compile, but I'm not sure how to write a
useful portable test case. Maybe later...
|
| |
|
|
|
|
| |
library.
|
|
|
|
| |
deletion of object from list argument during callout to fileno().
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Renamed
2. Several coding styles were being used here, owing to the multiple
contributors. I tried to convert everything to standard "python"
coding style for indentation, paren and brace placement, etc.
3. There were several potential error conditions that were never being
checked, and where I saw them, I added checks of return values,
etc. I'm pretty sure I got them all.
4. There were some old-style (pre PyArg_ParseTuple) argument
extraction and these were converted to use PyArg_ParseTuple.
All changes compile and run with the new test_select.py module, at
least on my Solaris/Sparc box.
|
|
|
|
|
| |
Note that since I have no idea how to test this, I didn't write a test
case. It does seem to at least compile on my system though.
|
| |
|
| |
|
|
|
|
|
|
| |
that's used in other parts of this module.
nis_maps(): use PyArg_NoArgs().
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Two interesting problems in nis_maplist(). First, it is possible that
clnt_create() will return NULL. This was being caught, but no Python
error was being set. I use clnt_spcreateerror() to generate the value
of the exception.
But why would clnt_create() fail? It's because no server was being
found. And why was this? It was because nis_maplist() tried only to
get the NIS master for the first map in the aliases list, which is
passwd.byname, and guess what? That's the one NIS map CNRI does *not*
export! So the yp_master() call was failing to return a valid
server. I now cycle through all the map aliases until I find a valid
master. If not, a different exception is set.
I'm not sure this is the completely correct way to do all this, but
short of rewriting the entire nismodule.c (to expose the proper API to
Python), it should do the trick.
|
|
|
|
|
| |
Note that there is no test suite for this module, and I don't plan to
write one just now.
|
| |
|
| |
|