summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* some macro cleanupsmoke-me/khw-macKarl Williamson2012-03-062-38/+11
|
* rework how the trie logic handles the newer EXACT nodetypesYves Orton2012-03-038-190/+262
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This cleans up and simplifies and extends how the trie logic interacts with the new node types. This change ultimately makes the EXACTFU, EXACTFU_SS, EXACTFU_NO_TRIE (renamed to EXACTFU_TRICKYFOLD) work properly with the trie engine regardless of whether the string is utf8 or latin1. This patch depends on the following: EXACT => utf8 or "binary" text EXACTFU => either pre-folded utf8, or latin1 that has to be folded as though it was utf8 EXACTFU_SS => special case of EXACTFU to handle \xDF/ss (affects latin1 treatment) EXACTFU_TRICKYFOLD => special case of EXACTFU to handle tricky non-latin1 fold rules EXACTF => "old style fold logic" untriable nodetype EXACTFA => (currently) untriable nodetype EXACTFL => (currently) untriable nodetype See the comments in regcomp.sym for these fold types. This patch involves a number of distinct, but related parts. Starting from compilation: * Simplify how we detect a triable sequence given the new nodetypes, this also probably fixed some "bugs" in how we detected certain sequences, like /||foo|bar/. * Simplify how we read EXACTFU nodes under utf8 by removing the now redundant folding logic (EXACTFU nodes under utf8 are prefolded). Also extend this logic to handle latin1 patterns properly (in conjunction with other changes) * Part of the problems associated with EXACTFU_SS and EXACTFU_TRICKYFOLD have to do with how the trie logic interacts with the minlen logic. This change handles both by pessimising the minlen when encounting these nodetypes. One observation is that the minlen logic is basically broken, and works only because it conflates bytes and codepoints in such a way that we more or less always get a value small enough that things work out anyway. Fixing that is properly is the job of another patch. * Part of the problem of doing folding under unicode rules is that there are a lot of foldings possible, some with strange rules. This means that the bitmap logic does not work correctly in all cases, as we currently do not have any way to populate it properly. So this patch disables the bitmap entirely when folding is involved until that is fixed. The end result of this is: we can TRIE/AHOCORASICK any sequence of EXACT, or EXACTFU (ish) nodes, regardless of utf8 or not, but we disable the bitmap when folding. A note for follow up relating to this patch is that the way EXACTFU_XXX nodes are currently dealt with we wont build the "maximal" trie because of their presence, instead creating a "jumptrie" consisting of either a leading EXACTFU node followed by a EXACTFU_XXX node, or vice versa. We should eventually address that.
* make test.pl show test number and name in failure diagnostics outputYves Orton2012-03-031-1/+1
| | | | | | | The old output would show only the line number as diagnostics but not the test number, nor the test name, which often contains very useful information. This patch makes sure this is visible in the diagnostics output of test failures.
* Sync Module-CoreList version in Maintainers.pl with CPANChris 'BinGOs' Williams2012-03-031-1/+1
|
* add "get rt.perl permissions" to RMGRicardo Signes2012-03-021-0/+7
|
* update release schedule for next releasersRicardo Signes2012-03-021-3/+3
|
* Added Dave Rolsky and Max Maischein to the KEEPERS OF THE PUMPKIN.Abigail2012-03-021-2/+2
|
* attributes.pm: Consistent spaces after dots in podFather Chrysostomos2012-03-011-7/+12
|
* attributes.pm: Add caveat about lvalue subsFather Chrysostomos2012-03-011-0/+9
|
* [perl #107366] Allow attributes to set :lvalue on defined subFather Chrysostomos2012-03-014-24/+31
| | | | | | | | | | | This provides enough rope for those who want to hang themselves, and also for those who know how to use the rope without hanging them- selves. :-) Since this is not generally a reliable thing to be doing, a warning is emitted whenever :lvalue is turned on or off on a defined subroutine. But attributes.pm will flip the flag anyway. :lvalue in a sub declar- ation still refuses to modify a defined Perl sub, as before.
* Add ambs to Maintainers.pl for ExtUtils::CBuilderDavid Golden2012-03-011-1/+2
|
* Stop the indexer from processing private modulesFlorian Ragwitz2012-02-292-7/+12
|
* perldata: Two spaces after dots, pleaseFather Chrysostomos2012-02-281-2/+2
|
* Explain why program must close __DATA__ handle.jkeenan2012-02-281-6/+7
| | | | Adapted from suggestion by David Golden++. For RT #37033.
* Bump Pod::Html version.Craig A. Berry2012-02-281-1/+1
|
* Portability tweaks to Pod::Html's cache.t.Craig A. Berry2012-02-281-3/+4
| | | | | | | | We need to unixify the current working directory since we're going to be comparing to the pod root that has been unixified internally in Pod::Html. Also clean up all versions of the generated files.
* Better cross-platform unixify for Pod::Html.Craig A. Berry2012-02-281-3/+25
| | | | | | | | | | This is mostly borrowed from CPANPLUS with additional tweaks to handle corner cases presented by the Pod::Html tests. It seems to work on VMS, Windows, and Mac OS X. Also tweak _save_page to make the call to ab2rel more robust in the case wherethe base is a special string indicating the current working directory ('./', '[]', or '.\') rather than a literal path.
* Different differences for Pod::Html tests.Craig A. Berry2012-02-281-10/+12
| | | | | Windows has FC (file compare), VMS has DIFFERENCES, and Linux is certainly not the only OS that can do unified diff.
* regcomp.c: Add commentKarl Williamson2012-02-281-0/+2
|
* regcomp.c: Add comment, reorder #define more logicallyKarl Williamson2012-02-281-1/+3
|
* re/pat.t: Remove obsolete commentKarl Williamson2012-02-281-3/+1
|
* (?foo:...) loses passed in charsetKarl Williamson2012-02-282-4/+16
| | | | | | This commit looks for the passed-in charset, and overrides it only if it is /d and the pattern requires /u. Previously the passed-in value was ignored.
* Patch [perl #111400] [:upper:] broken for above Latin1Karl Williamson2012-02-283-11/+51
| | | | | | | | | | | | This was an off-by-one error caused by my failing to realize that things had to be done differently at the 255/256 boundary depending on whether U+00FF matched or did not match the property. Two properties were affected, [:upper:] and [:punct:]. The bug was that all code points above the first one > 255 that legitimately matches the property will match whether or not they should. In the case of [:upper:], this meant that effectively anything from 256..infinity matched. For [:punct:], it was anything above U+037D.
* Corion's release is done. Check him offJesse Vincent2012-02-281-1/+1
|
* av_fetch: de-duplicate small bit of codeDavid Mitchell2012-02-271-11/+5
| | | | | | | | | | | | | | make the code slightly smaller by changing if (A) return X; if (B) return X; into ` if (A || B) return X;
* Merge the fixes for RT #37033 into blead.Nicholas Clark2012-02-279-34/+86
|\
| * The parser should always close the file handle that it opened.Nicholas Clark2012-02-276-7/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously it would leave the file handle open if it was (equal to) stdin, on the assumption that this must have been because no script name was supplied on the interpreter command line, so the interpreter was defaulting to reading the script from standard input. However, if the program has closed STDIN, then the next file handle opened (for any reason) will have file descriptor 0. So in this situation, the handle that require opened to read the module would be mistaken for the above situation and left open. Effectively, this leaked a file handle. This is now fixed, by explicitly tracking from parser creation time whether it should keep the file handle open, and only setting this flag when defaulting to reading the main program from standard input. This resolves RT #37033.
| * Move the close-on-exec logic to one place, at the end of S_open_script().Nicholas Clark2012-02-271-10/+4
| | | | | | | | | | | | Now that the logic for stdin is implemented as an early return of NULL from S_open_script(), in all cases that reach the end of S_open_script(), rsfp is non-NULL, and a file handle that we wish to set to close on exec.
| * Change S_open_script() to return NULL to signal "read from stdin".Nicholas Clark2012-02-271-1/+4
| | | | | | | | | | Move the logic to assign PerlIO_stdin() to rsfp from S_open_script() to its only caller, S_parse_body().
| * In perl.c, change S_open_script() to return rsfp.Nicholas Clark2012-02-274-20/+19
| | | | | | | | | | | | Previously it was being passed &rsfp as a parameter, because it was returning another value, fdscript. However, the return value has been ignored since commit cc69b689ee7c2745 removed suidperl in January 2009.
| * In struct yy_parser, change lex_flags to a U8, from part of a bitfield.Nicholas Clark2012-02-271-1/+1
|/ | | | | | | | | | lex_flags holds 4 flag bits, with multiple flag bits manipulated together at times, so they can't be split out into individual bitfields. This change permits the C compiler to generate simpler code, reducing toke.o by about 400 bytes on this platform, but doesn't change the size of the structure. lex_flags was added in commit 802a15e9c01d1a0b in August 2011, so is not in any stable release.
* Improve the tests for -xNicholas Clark2012-02-274-2/+22
| | | | | Test the error message generated when -x can't find a "#!perl" line. Test that this error message still appears when -x is used with -e.
* Improve the tests for the -n and -p switches.Nicholas Clark2012-02-272-4/+17
| | | | | | Verify that -p actually runs the code in the program body. Verify that -n doesn't implicitly print out the contents of $_. For both, verify that an END block runs after the implicit loop.
* Simplify platform specific code in t/run/cloexec.tNicholas Clark2012-02-271-6/+3
| | | | | | | | | | | | | Don't assign to two lexical variables, $Is_VMS and $Is_Win32, only to use them immediately for the same purpose - to skip the entire test. In turn, there's no need to conditionally set $quote to a value suitable for VMS or Win32, when neither OS ever runs the test. The code has been this way since the file was added by commit 742218b34f58f961 in Nov 2006. Hence I don't think that the vestigial $quote logic corresponds to pre-commit version that did run on these platforms. Instead I infer that it has come from t/op/exec.t, used as a template for running sub-scripts in a portable fashion.
* perldiag: Reflow for better splain outputFather Chrysostomos2012-02-261-3/+3
|
* Add an example how to fix '%s: command not found'Max Maischein2012-02-261-2/+5
|
* Increase $attributes::VERSION to 0.18Father Chrysostomos2012-02-261-1/+1
|
* doco improvement for attributes.pmDavid Cantrell2012-02-261-0/+16
| | | | | | It seems that many people have trouble understanding how to add custom attributes to their subroutines. Here's a doc patch that will hopefully make things clearer:
* Make CPAN upstream for File::TempFather Chrysostomos2012-02-251-1/+1
| | | | | | | | | It was set to undef, which meant it hadn’t been discussed. In actuality, it is actively maintained on CPAN. See, for instance: https://rt.cpan.org/Ticket/Display.html?id=75077#txn-1038945 So this brings Maintainers.pl closer in line with reality.
* Don’t ‘normalise’ hints for bare ‘no feature’Father Chrysostomos2012-02-252-10/+10
| | | | | | | | | ‘Normalise’ in this case means to set $^H to indicate that features are in %^H (FEATURE_BUNDLE_CUSTOM) and to make %^H contain the current feature set. Since ‘no feature’ sets the default feature bundle in $^H, this is unnecessary in that case.
* regen/feature.pl: Show perl.h line num in error msgFather Chrysostomos2012-02-251-1/+1
|
* perlfunc: bad wordingFather Chrysostomos2012-02-251-1/+1
| | | | | It is not features not in the current version, but those not in the requested version, that are disabled.
* arybase.xs: Wrap PL_check safelyFather Chrysostomos2012-02-251-2/+2
|
* Increase $arybase::VERSION to 0.05Father Chrysostomos2012-02-251-1/+1
|
* don't taint $$ determined by getpid()Zefram2012-02-252-2/+12
| | | | | | Reading $$ in a tainted expression was tainting the internal sv_setiv() on $$. Since the value being set came directly from getpid(), it's always safe, so override the tainting there. Fixes [perl #109688].
* update perlfunc.html generatorZefram2012-02-251-30/+15
| | | | | | The format of the individual function HTML files has changed. The index generator needs to update to successfully extract the NAME sections. Fixes [perl #107870].
* delay allocating trans table until neededZefram2012-02-254-11/+19
| | | | | | | | | | | | | | | | | | | Previously, a table was being allocated for OP_TRANS(|R), in a PVOP arrangement, as soon as the op was built. However, it wasn't used immediately, and for UTF8-flagged ops it would be thrown away, replaced by an SV-based translation table in a SVOP or PADOP arrangement. This mutation of the op structure occurred in pmtrans(), some time after original op building. If an error occurred before pmtrans(), requiring the op to be freed, op_clear() would be misled by the UTF8 flags into treating the PV as an SV or pad index, causing crashes in the latter case [perl #102858]. op_clear() was implicitly assuming that pmtrans() had been performed, due to lacking any explicit indication of the op's state of construction. Now, the PV table is allocated by pmtrans(), when it's actually going to populate it. The PV doesn't get allocated at all for UTF8-flagged ops. Prior to pmtrans(), the op_pv/op_sv/op_padix field is all bits zero, so there's no problem with freeing the op.
* Update the link for Module-Build/core integrationChris 'BinGOs' Williams2012-02-241-1/+1
|
* mktables: Correct Unicode 6.1 omissionKarl Williamson2012-02-241-0/+4
| | | | | | Unicode 6.1 erroneously omitted Takri as a script that uses two characters, and have voted to publish the correction that this patch makes. There isn't an official Corrigendum yet.
* correct a Pod foible that broke bleadRicardo Signes2012-02-241-1/+1
|
View Lorry Controller Status