Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Perl: porters

[perl #114320] add RT #114144 to maint 5.16

 

 

Perl porters RSS feed   Index | Next | Previous | View Threaded


perlbug-followup at perl

Jul 29, 2012, 7:54 PM

Post #1 of 2 (112 views)
Permalink
[perl #114320] add RT #114144 to maint 5.16

# New Ticket Created by bulk 88
# Please include the string: [perl #114320]
# in the subject line of all future correspondence about this issue.
# <URL: https://rt.perl.org:443/rt3/Ticket/Display.html?id=114320 >




This is a bug report for perl from bulk88 [at] hotmail,
generated with the help of perlbug 1.39 running under perl 5.17.2.


-----------------------------------------------------------------
[Please describe your issue here]

Would it be possible to add the 2 patches (1 has been applied to blead, the other hasn't been), in https://rt.perl.org/rt3/Ticket/Display.html?id=114144 (and a regen.pl) to the next 5.16?

I categorize as a portability fix, and they do not affect ABI since Win32 GCC Perls have been compiled with noreturn declaration for years, code after a longjmp is unreachable by definition. Perl claims to be ABI compatible Visual C Perl and Mingw GCC XS modules, see http://www.perlmonks.org/?node_id=614808 , the other direction I am not sure about.

It will reduce about a dozen KB (in my experience) of unreachable machine code in the perl dll, and also reduce the size of any XS code compiled Visual C.

I propose this because 5.18 is 10 months away, and 5.16 still has 21 months on it.

[Please do not change anything below this line]
-----------------------------------------------------------------
---
Flags:
    category=core
    severity=low
---
Site configuration information for perl 5.17.2:

Configured by Owner at Sat Jul 21 05:38:53 2012.

Summary of my perl5 (revision 5 version 17 subversion 2) configuration:
  
  Platform:
    osname=MSWin32, osvers=5.1, archname=MSWin32-x86-multi-thread
    uname=''
    config_args='undef'
    hint=recommended, useposix=true, d_sigaction=undef
    useithreads=define, usemultiplicity=define
    useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef
    use64bitint=undef, use64bitall=undef, uselongdouble=undef
    usemymalloc=n, bincompat5005=undef
  Compiler:
    cc='cl', ccflags ='-nologo -GF -W3 -MD -Zi -DNDEBUG -O1 -GL -DWIN32 -D_CONSOLE -DNO_STRICT  -DPERL_TEXTMODE_SCRIPTS -DPERL_IMPLICIT_CONTEXT -DPERL_IMPLICIT_SYS -DUSE_PERLIO -D_USE_32BIT_TIME_T',
    optimize='-MD -Zi -DNDEBUG -O1 -GL',
    cppflags='-DWIN32'
    ccversion='13.10.6030', gccversion='', gccosandvers=''
    intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234
    d_longlong=undef, longlongsize=8, d_longdbl=define, longdblsize=8
    ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='__int64', lseeksize=8
    alignbytes=8, prototype=define
  Linker and Libraries:
    ld='link', ldflags ='-nologo -nodefaultlib -debug -opt:ref,icf -ltcg  -libpath:"c:\perl517\lib\CORE"  -machine:x86'
    libpth="C:\Program Files\Microsoft Visual Studio .NET 2003\VC7\lib"
    libs=oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib  comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib  netapi32.lib uuid.lib ws2_32.lib mpr.lib winmm.lib  version.lib odbc32.lib odbccp32.lib comctl32.lib msvcrt.lib
    perllibs=oldnames.lib kernel32.lib user32.lib gdi32.lib winspool.lib  comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib  netapi32.lib uuid.lib ws2_32.lib mpr.lib winmm.lib  version.lib odbc32.lib odbccp32.lib comctl32.lib msvcrt.lib
    libc=msvcrt.lib, so=dll, useshrplib=true, libperl=perl517.lib
    gnulibc_version=''
  Dynamic Linking:
    dlsrc=dl_win32.xs, dlext=dll, d_dlsymun=undef, ccdlflags=' '
    cccdlflags=' ', lddlflags='-dll -nologo -nodefaultlib -debug -opt:ref,icf -ltcg  -libpath:"c:\perl517\lib\CORE"  -machine:x86'

Locally applied patches:
   

---
@INC for perl 5.17.2:
    C:/perl517/site/lib
    C:/perl517/lib
    .

---
Environment for perl 5.17.2:
    HOME (unset)
    LANG (unset)
    LANGUAGE (unset)
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=C:\perl517\bin;C:\Program Files\Microsoft Visual Studio .NET 2003\Common7\IDE;C:\Program Files\Microsoft Visual Studio .NET 2003\VC7\BIN;C:\Program Files\Microsoft Visual Studio .NET 2003\Common7\Tools;C:\Program Files\Microsoft Visual Studio .NET 2003\Common7\Tools\bin\prerelease;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\system32\wbem;
    PERL_BADLANG (unset)
    SHELL (unset)


jand at activestate

Jul 30, 2012, 6:22 PM

Post #2 of 2 (104 views)
Permalink
RE: [perl #114320] add RT #114144 to maint 5.16 [In reply to]

On Sun, 29 Jul 2012, bulk 88 (via RT) wrote:
>
>
> Would it be possible to add the 2 patches (1 has been applied to
> blead, the other hasn't been), in

Thanks for mentioning the 2nd patch; I never saw email notifications
that the bug had be re-opened.

> https://rt.perl.org/rt3/Ticket/Display.html?id=114144 (and a regen.pl)
> to the next 5.16?
>
> I categorize as a portability fix, and they do not affect ABI since
> Win32 GCC Perls have been compiled with noreturn declaration for
> years, code after a longjmp is unreachable by definition. Perl claims
> to be ABI compatible Visual C Perl and Mingw GCC XS modules, see
> http://www.perlmonks.org/?node_id=614808 , the other direction I am
> not sure about.
>
> It will reduce about a dozen KB (in my experience) of unreachable
> machine code in the perl dll, and also reduce the size of any XS code
> compiled Visual C.
>
> I propose this because 5.18 is 10 months away, and 5.16 still has 21
> months on it.

I don't see how this could possibly count as a portability fix; having
a few KB of unreachable code is not a serious issue at all in my mind,
so it does not fit the spirit of pod/perlpolicy.pod.

Cheers,
-Jan

Perl porters RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.