summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Make COW-clobbering fasterFather Chrysostomos2011-11-241-3/+8
| | | | | | | | | | | | | | | | | | | | | | | There is this special SV_COW_DROP_PV flag that gets passed to sv_force_normal_flags to signal that the SV is about to be clobbered, so there is no point in allocating a new PV. But this flag was not actually being used, until the previous commit commandeered it for globs (despite the name). Now it actually does what it says. Before and after: $ time ./perl -e '$x = __PACKAGE__, undef $x for 1..8000000' real 0m5.758s user 0m5.740s sys 0m0.008s $ time ./perl -e '$x = __PACKAGE__, undef $x for 1..8000000' real 0m3.290s user 0m3.282s sys 0m0.006s
* Make assignment over glob copies much fasterFather Chrysostomos2011-11-244-7/+9
| | | | | | | | | | | | | | | | | | | sv_force_normal is passed the SV_COW_DROP_PV flag if the scalar is about to be written over. That flag is not currently used. We can speed up assignment over fake GVs a lot by taking advantage of the flag. Before and after: $ time ./perl -e '$x = *foo, undef $x for 1..2000000' real 0m4.264s user 0m4.248s sys 0m0.007s $ time ./perl -e '$x = *foo, undef $x for 1..2000000' real 0m1.820s user 0m1.812s sys 0m0.005s
* Call FETCH once for rcatlineFather Chrysostomos2011-11-242-4/+10
|
* lib/version.t: suppress warningFather Chrysostomos2011-11-241-1/+1
|
* perlfunc: Don’t put __SUB__ between substr entriesFather Chrysostomos2011-11-241-3/+3
| | | | | I made the mistake of putting it after the first =item substr, instead of before it.
* sysread should not ignore magic on its bufferFather Chrysostomos2011-11-242-1/+26
| | | | | | sysread uses SvPV_force, which has a bug in it. Even if the SV_GMAGIC flag is passed to sv_pvn_force_flags (which SvPV_force does), it ignores magic in any typeglob argument.
* Make sselect call fetch onceFather Chrysostomos2011-11-243-3/+22
| | | | | | | Not only does this commit make four-argument select call fetch once on each argument (instead of sometimes 0 times), but it also checks whether the argument is a string after calling fetch now, instead of before, in determining whether to warn about a non-string.
* Move substr tests under t/opFather Chrysostomos2011-11-243-2/+2
| | | | | Commit a4499558 moved them under t/re along with the subst tests, but these are unrelated to regexps.
* Call FETCH once when chomping a tied refFather Chrysostomos2011-11-242-1/+6
|
* pp.c: Remove useless read-only check from S_do_chompFather Chrysostomos2011-11-241-1/+1
| | | | | | After sv_force_normal_flags, the scalar will no longer be read-only, except in those cases where sv_force_normal_flags croaks. So this check will never be true when SvFAKE was true.
* Call FETCH once for $tied_ref =~ y/a/b/Father Chrysostomos2011-11-242-2/+4
|
* Increase $IO::File::VERSION to 1.16Father Chrysostomos2011-11-241-1/+1
|
* Remove ‘use File::Spec’ from IO::FileFather Chrysostomos2011-11-241-1/+0
| | | | It is not using it any more.
* __SUB__ should warn in void contextFather Chrysostomos2011-11-242-0/+4
|
* Use correct err msg in XS version checkFather Chrysostomos2011-11-242-2/+9
| | | | | | | | | | When an XS module’s version is checked when it is loading, the string "version" should be treated the same way as "versions" and emit the ‘Invalid version format’ error, instead of being treated as a version object at first and then rejected by the validator with the ‘Invalid version object’ error. See also perl #102586.
* Remove $SIG{__WARN__} from XSLoader.tFather Chrysostomos2011-11-241-3/+0
| | | | Nothing is using the results of this handler any more.
* Produce right error msg for $ver < "version"Father Chrysostomos2011-11-242-1/+6
| | | | | | | | | | | | | | "version" was being treated as a version object and then failing the validation check. It should be treated as a string, just like "versions": $ perl5.15.4 -Ilib -e '$^V < "version"' Invalid version object at -e line 1. $ perl5.15.4 -Ilib -e '$^V < "versions"' Invalid version format (dotted-decimal versions require at least three parts) at -e line 1. See also perl #102586.
* gv.c: Remove SV_GMAGIC from sv_catpvn_flags call.Father Chrysostomos2011-11-241-1/+1
| | | | This function doesn’t take that flag. It wasn’t doing anything.
* Update Pod-LaTeX to CPAN version 0.60Chris 'BinGOs' Williams2011-11-234-9/+23
| | | | | | | | | | [DELTA] added another LaTeX escape: --- => -{}-{}- Pod::LaTeX doesn't handle -- in PODs specially, passing it directly to LaTeX, which then proceeds to replace it with a single -. This patch replaces ----- with -{}-{}-{}-{}-
* Update Unicode-Collate to CPAN version 0.86Chris 'BinGOs' Williams2011-11-2320-85/+5233
| | | | | | | | | [DELTA] 0.86 Wed Nov 23 17:16:00 2011 - tailored compatibility ideographs as well as unified ideographs for the locales: ja, ko, zh__big5han, zh__gb2312han, zh__pinyin, zh__stroke. - added loc_cjkc.t in t.
* Move the implementation of --validate from bisect.pl to bisect-runner.plNicholas Clark2011-11-232-4/+5
| | | | | | | | | | | --validate sets a default testcase by assigning to @ARGV if its empty. It makes more sense to do this in bisect-runner.pl, as that processes options fully, unlike bisect.pl, which passes most through. Hence bisect.pl doesn't know if some elements of @ARGV are actually options, and hence no testcase has been supplied, and hence the default is needed. This change permits the use of --validate with build options such as -D to work as expected.
* Add a --make option to bisect.pl, to specify an alternate make command.Nicholas Clark2011-11-231-3/+20
| | | | For example, one can utilise this to use gmake instead of the system make.
* UNIVERSAL::VERSION should treat "version" as a stringFather Chrysostomos2011-11-232-3/+10
| | | | | | | | | | | | | It was treating it as a version object and then failing the validation test, instead of treating it as an invalid version format, as it does with "versions": $ ./perl -Ilib -e'$VERSION = "versions"; main->VERSION(1)' Invalid version format (dotted-decimal versions require at least three parts) at -e line 1. $ ./perl -Ilib -e'$VERSION = "version"; main->VERSION(1)' Invalid version object at -e line 1. See also perl #102586.
* printf "%vd", "version" should not SEGVFather Chrysostomos2011-11-232-1/+2
| | | | See perl #102586.
* On AIX, bisect-runner.pl must patch Makefile.SH for parallel make issues.Nicholas Clark2011-11-231-0/+15
| | | | | It needs to emulate the bug fix of e6807d8ab22b761c, to ensure lib/Config.pm is built before makedef.pl is run.
* makedef.pl uses Config.pm, so needs a Makefile dependency on it.Nicholas Clark2011-11-231-2/+2
| | | | | | | | Commit 9d6c7f2eccef26a6 changed makedef.pl to use Config.pm Commit 208d7614b345d1fb refactored makedef.pl to read values from %Config::Config, instead of reading directly from config.sh Hence we can replace the dependency on config.sh with one on lib/Config.pm
* [perl #102586] version->new("version") SEGVsFather Chrysostomos2011-11-223-2/+16
| | | | | | This adds an ROK check after calling sv_derived_from, as the latter also works for class names. It is done after sv_derived_from, rather than before, as sv_derived_from calls get-magic.
* arybase.xs be more defensiveReini Urban2011-11-221-0/+4
| | | | | | There can be no other keys used with ab_ck_base but clang's static analyzer has a point in complaining about the missing default case. This is too fragile if any CHECK is added in BOOT.
* Correct spelling of double free warningFather Chrysostomos2011-11-222-2/+2
| | | | | | | The perldiag entry said ‘nonexistent’, which is correct. hv.c said ‘non-existent’, which is, well, questionable. They should be the same, so I corrected hv.c. I also added the %s%s to the end in perldiag.
* Correct perldiag entry for sv_replace panicFather Chrysostomos2011-11-221-5/+5
| | | | | | | | | | | | | | | This commit changed the warning to an error and reworded it, but never updated the docs (this commit completes the TODO): perl-5.8.0-5715-g30e5c35 commit 30e5c352c9c1099120007e8b6e9318a33d99b3bb Author: Nicholas Clark <nick@ccl4.org> Date: Thu Aug 25 13:46:31 2005 +0000 Promote the warning about reference miscount in sv_replace to a panic. TODO - document the panics p4raw-id: //depot/perl@25330
* perldiag: Fix categories of internal severe warningsFather Chrysostomos2011-11-221-5/+5
| | | | | Some severe/default warnings in the internal and debugging categories were listed as errors (P) or regular warnings (W).
* sv.c/sv_insert_flags: typoFather Chrysostomos2011-11-221-1/+2
|
* amagic_deref_call does not necessitate SPAGAINFather Chrysostomos2011-11-223-7/+0
| | | | | As amagic_deref_call pushes a new stack, PL_stack_sp will always have the same value before and after, so SPAGAIN is unnecessary.
* [Merge] [RT #36079] Convert ` to 'Father Chrysostomos2011-11-2280-412/+412
|\
| * [RT #36079] Convert ` to '.jkeenan2011-11-222-6/+6
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-226-78/+78
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-224-37/+37
| |
| * [RT #36079] Where sensible, replace consecutive single quotation marks with ↵jkeenan2011-11-223-12/+12
| | | | | | | | a double quotation mark.
| * [RT #36079] Due to test failures, revert conversion from backtick to single ↵jkeenan2011-11-221-2/+2
| | | | | | | | quote.
| * [RT #36079] Convert ` to '.jkeenan2011-11-221-1/+1
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-224-8/+8
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-221-2/+2
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-222-2/+2
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-225-8/+8
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-222-2/+2
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-222-2/+2
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-221-1/+1
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-221-1/+1
| |
| * [RT #36079] Convert ` to '.jkeenan2011-11-2211-42/+42
| |
| * Version bumpsif-0.0602Father Chrysostomos2011-11-2218-18/+18
| |
View Lorry Controller Status