[perl #39760] make warnings (r13197 - x86-msvc-7.1)

[Jerry Gay]

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

compilers\imcc\imcparser.c
imcparser.c
compilers\imcc\imcparser.c(878) : warning C4273: 'malloc' :
inconsistent dll linkage
compilers\imcc\imcparser.c(885) : warning C4273: 'free' : inconsistent
dll linkage

compilers\ast\astparser.c
astparser.c
compilers\ast\astparser.c(1298) : warning C4102: 'yyerrlab1' :
unreferenced label

compilers\imcc\main.c
main.c
compilers\imcc\main.c(480) : warning C4273: 'imcc_init' : inconsistent
dll linkage

~jerry

[Ron Blaschke]

In the current head (r13307) imcparser.c seems to be mostly fine.

compilers\imcc\imcparser.c
imcparser.c
compilers/imcc/imcparser.c(4098) : warning C4102: 'yyerrlab1' :
unreferenced label

But compilers/imcc/main.c still suffers.

compilers\imcc\main.c
main.c
compilers\imcc\main.c(480) : warning C4273: 'imcc_init' :
inconsistent dll linkage

Attached patch brings the declarations of imcc_init in main.c and imc.h
in sync.

compilers\imcc\main.c
main.c

Ron

[Leopold Toetsch]

Am Samstag, 15. Juli 2006 21:27 schrieb Ron Blaschke:

> Attached patch brings the declarations of imcc_init in main.c and imc.h
> in sync.

Well, much simpler - I've deleted the line in main.c :-)
Thanks for the hint.

> Ron

leo

[Ron Blaschke]

Bummer, should have given the patch more thought. The clue was only a
few lines up: C<#include "imc.h">. Sorry about that, and thanks for
fixing it.

> leo

Ron

[Ron Blaschke]

Jerry Gay (via RT) wrote:

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

> compilers\ast\astparser.c
> astparser.c
> compilers\ast\astparser.c(1298) : warning C4102: 'yyerrlab1' :
> unreferenced label

compilers/ast/astparser.c is generated by GNU Bison 1.875. I have GNU
Bison 2.1 installed, which generates code that doesn't yield any warning.

bison -v -y compilers\ast\ast.y -d -o compilers\ast\astparser.c
-p AST
C:\Perl\bin\perl.exe -MExtUtils::Command -e touch
compilers\ast\ast.y.flag compilers\ast\astparser.c
compilers\ast\astparser.h
bison -v -y compilers\ast\ast.y -d -o compilers\ast\astparser.c
-p AST
C:\Perl\bin\perl.exe -MExtUtils::Command -e touch
compilers\ast\ast.y.flag compilers\ast\astparser.c
compilers\ast\astparser.h
compilers\ast\astparser.c
astparser.c

Is that sufficient to close this issue, or should I provide a patch to
disable the C4102 warning?

Ron

[Paul Cochrane via RT]

Is this ticket still relevant? Are we still seeing these warnings with
msvc?

Paul