3 * > 3. In several cases, "perl ppport.h --copy=.new" output a new file in
4 > which the only change was the addition of "#include "ppport.h"". In each
5 > case, that actually wasn't necessary because the source file in question
6 > already #included another source file which #included ppport.h itself.
7 > Would it be possible for the analyzer to follow #include directives to
8 > spot cases like this?
10 Uh, well, I guess it would be possible. But I have some concerns:
12 1. ppport.h is already too big. :-)
14 2. There is code in ppport.h to actually remove an
18 if it appears not to be needed. If it's not needed in your
19 included file, it might be dropped from there and moved to
20 the other file that included the first one. This would make
21 the logic much more complicated.
23 3. As ppport.h is configurable, it's not (always) a good idea
24 to put it into a file that's included from another file.
26 I guess I'll have to think about this a little more. Maybe I can
27 come up with a fancy solution that doesn't increase the code size
31 * On 14/12/06, Nicholas Clark <nick@ccl4.org> wrote:
32 > On Thu, Dec 14, 2006 at 05:03:24AM +0100, Andreas J. Koenig wrote:
34 > > Params::Validate and Clone suffer from the same cold:
36 > The same patch will make both compile and pass tests.
37 > I'm wondering if it might be better to totally drop SVt_PBVM and let source
38 > code fail to compile.
40 I don't think so. Because :
41 1. your redefinition of SVt_PBVM is probably what most XS modules want
42 2. anyway, if we remove it from the core, it might appear in Devel::PPPort :)
45 * maybe backport bytes_from_utf8() for 5.6.0 (or even before)?
47 * check which of the following we need to support:
318 * have an --env option for soak to set env variable combinations
320 * only overwrite generated files if they actually changed
322 * try to make parts/apicheck.pl automatically find NEED_ #defines
324 * add support for my_vsnprintf?
326 * try to perform some core consistency checks:
328 - check if 'd' flag in embed.fnc matches with
329 supplied documentation
331 - check if all public API is documented
333 * check (during make regen?) if MAX_PERL in PPPort_pm.PL
336 * see if we can implement sv_catpvf() for < 5.004
340 * improve apicheck (things like utf8_mg_pos_init() are
341 not currently checked)
343 * more documentation, more tests
345 * Resolve dependencies in Makefile.PL and remind of