Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Verilog-Perl CPAN distribution.

Report information
The Basics
Id: 25833
Status: resolved
Worked: 5 min
Priority: 0/
Queue: Verilog-Perl

People
Owner: wsnyder [...] wsnyder.org
Requestors: nodine [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 2.341
  • 2.372
Fixed in: (no value)



Subject: Fails to parse first statement after null port list
MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1174947642-19780-4"
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 387
Download (untitled) / with headers
text/plain 387b
The Verilog::SigParser never calls the signal_decl routine for the declaration of reg "a". The problem is that $self->{is_signal_ok} is false at the time the semicolon after the register declaration is processed. This can be traced back to the line that says # Fallthru, more ; prep for next command is below which does not, in fact, fall through to the rest of the ';' processing.
Subject: addsr.v
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="----------=_1174947642-19780-3"
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Length: 0
Content-Type: text/plain; charset="utf8"
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 0
Content-Type: application/tkgate; name="addsr.v"
Content-Disposition: inline; filename="addsr.v"
Content-Transfer-Encoding: base64
Content-Length: 1253
Download addsr.v
application/tkgate 1.2k

Message body not shown because it is not plain text.

MIME-Version: 1.0
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Message-Id: <rt-3.6.HEAD-19775-1175199596-1983.25833-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 613
Download (untitled) / with headers
text/plain 613b
On Mon Mar 26 18:20:45 2007, NODINE wrote: Show quoted text
> The Verilog::SigParser never calls the signal_decl routine for the > declaration of reg "a". The problem is that $self->{is_signal_ok} is > false at the time the semicolon after the register declaration is > processed. This can be traced back to the line that says
I agree that's probably a problem, but the only place is_signal_ok is set to 0 looks to be dead code. When I run your verilog code using a hacked version of t/35_sigparser.t I see all of the signals. Can you send me an example script that is broken? BTW I'm sorry the parser isn't bison'ized yet.
MIME-Version: 1.0
X-Spam-Status: No, hits=-2.6 required=8.0 tests=BAYES_00,DK_POLICY_SIGNSOME,SPF_HELO_PASS,SPF_PASS
In-Reply-To: <rt-3.6.HEAD-19775-1175199596-1983.25833-6-0 [...] rt.cpan.org>
Received-SPF: pass (x1.develooper.com: domain of nodine [...] intrinsity.com designates 208.123.79.101 as permitted sender)
References: <RT-Ticket-25833 [...] rt.cpan.org> <rt-3.6.HEAD-19775-1175199596-1983.25833-6-0 [...] rt.cpan.org>
Reply-To: nodine [...] intrinsity.com
Content-Type: text/plain; charset=UTF-8; format=flowed
Organization: Intrinsity, Inc.
X-RT-Original-Encoding: utf-8
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 3F9D64D80A2 for <bug-Verilog-Perl [...] rt.cpan.org>; Thu, 29 Mar 2007 16:43:10 -0400 (EDT)
Received: (qmail 27690 invoked by alias); 29 Mar 2007 20:43:10 -0000
Received: from gw.intrinsity.com (HELO mailhost.intrinsity.com) (208.123.79.101) by la.mx.develooper.com (qpsmtpd/0.28) with ESMTP; Thu, 29 Mar 2007 13:43:08 -0700
Received: from victoria.intrinsity.com (mailhost.eng.intrinsity.com [192.168.1.29]) by mailhost.intrinsity.com (Postfix) with ESMTP id 2683218C004 for <bug-Verilog-Perl [...] rt.cpan.org>; Thu, 29 Mar 2007 15:43:03 -0500 (CDT)
Received: from [192.168.1.44] (cardiff.eng.intrinsity.com [192.168.1.44]) by victoria.intrinsity.com (Postfix) with ESMTP id E9862AC01 for <bug-Verilog-Perl [...] rt.cpan.org>; Thu, 29 Mar 2007 15:43:02 -0500 (CDT)
Delivered-To: cpan-bug+verilog-perl [...] diesel.bestpractical.com
Subject: Re: [rt.cpan.org #25833] Fails to parse first statement after null port list
User-Agent: Thunderbird 1.5 (X11/20051201)
Return-Path: <nodine [...] intrinsity.com>
X-Spam-Check-BY: la.mx.develooper.com
X-Original-To: bug-Verilog-Perl [...] rt.cpan.org
Date: Thu, 29 Mar 2007 15:43:02 -0500
Message-Id: <460C24D6.7020302 [...] intrinsity.com>
To: bug-Verilog-Perl [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Mark Nodine <nodine [...] intrinsity.com>
X-RT-Original-Encoding: utf-8
RT-Message-ID: <rt-3.6.HEAD-19730-1175201368-1489.25833-0-0 [...] rt.cpan.org>
Content-Length: 893
Download (untitled) / with headers
text/plain 893b
via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=25833 > > > On Mon Mar 26 18:20:45 2007, NODINE wrote:
>> The Verilog::SigParser never calls the signal_decl routine for the >> declaration of reg "a". The problem is that $self->{is_signal_ok} is >> false at the time the semicolon after the register declaration is >> processed. This can be traced back to the line that says
> > I agree that's probably a problem, but the only place is_signal_ok is > set to 0 looks to be dead code. > > When I run your verilog code using a hacked version of t/35_sigparser.t > I see all of the signals. Can you send me an example script that is broken?
When I worked on a simple example, it was fine. When I ran my full program using perl 5.8.8 instead of 5.8.0, it was fine. The full script failed on 5.8.0. So I guess you shouldn't worry about it. Sorry to trouble you. --Mark
MIME-Version: 1.0
In-Reply-To: <rt-3.6.HEAD-19730-1175201368-1489.25833-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.418 (Entity 5.418)
Content-Disposition: inline
Message-Id: <rt-3.6.HEAD-16474-1175606025-1673.25833-0-0 [...] rt.cpan.org>
References: <RT-Ticket-25833 [...] rt.cpan.org> <rt-3.6.HEAD-19775-1175199596-1983.25833-6-0 [...] rt.cpan.org> <460C24D6.7020302 [...] intrinsity.com> <rt-3.6.HEAD-19730-1175201368-1489.25833-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Original-Encoding: utf-8
Content-Length: 42
I've released 2.373 which fixes this bug.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.