Jump to content

Recommended Posts

Hi,

Based on Accellera's recently released standard, I am now developing a PSS front-end. While doing this, I came across several grammar problems, and also some typos in the example code given in the document. In this email, I have attached 2 text documents:

(1) error_grammar.txt : a list of grammar problems, and their suggested fixes.

(2) error_examples.txt : a list of typo mistakes in the examples.

The page numbers mentioned are the pdf page numbers, NOT the printed page numbers.

I would like to hear from the others if these mentioned fixes are good, or other changes are required.

Regards.. Raj S Mitra, rsm@verikwest.com

error_examples.txt

error_grammar.txt

Share this post


Link to post
Share on other sites

Hello Raj,

    Many thanks for your review comments in the grammar and examples! Your comments regarding unused terminals (issues 2, 4, 5, 6, 12, 13, 14) make perfect sense, and I'll proceed to file Mantis items for these. I will need to investigate a bit more on the issues you point out with respect to items that appear incorrectly grouped, but will also develop proposals to add to Mantis items for these as well.

 

Best Regards,

Matthew

 

Share this post


Link to post
Share on other sites
On 7/26/2017 at 10:04 AM, Matthew Ballance said:

Hello Matthew,

Thanks for your review of my comments. You can always reach me at my emails below, for any detailed discussion.

There are some other problems with the grammar as well, eg. no way to define a constant/parameter, only 2-dim arrays supported, etc. But in my email I had not focused on missing features. I like a light-weight language, but some basic features like these may be needed to make it more flexible.

Regards.. Raj, rsm@verikwest.com, rsmitra@gmail.com

 

Quote

 

 

Share this post


Link to post
Share on other sites

Section B.10 -- Is the syntax for "expression_constraint_item" correct (line 15 of page 202)?  This appears to state that an implicand_constraint_item is required to accompany an expression...

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×