Modify

Opened 5 years ago

Last modified 4 months ago

#1523 accepted enhancement

Use Piraha to Parse all CCL files

Reported by: Steven R. Brandt Owned by: Steven R. Brandt
Priority: optional Milestone: Cactus_4.4.0
Component: Cactus Version: development version
Keywords: Cc:

Description

Piraha should be used to parse all the CCL files. This would give us a well-defined grammar and the ability to let other tools use the CCL files in a reliable way.

Attachments (0)

Change History (11)

comment:1 Changed 5 years ago by Steven R. Brandt

Owner: set to Steven R. Brandt
Status: newaccepted

comment:2 Changed 3 years ago by Frank Löffler

Milestone: Cactus_4.3.0Cactus_4.4.0

comment:3 Changed 3 years ago by Frank Löffler

Is this going anywhere?

comment:4 Changed 2 years ago by Steven R. Brandt

Last edited 2 years ago by Steven R. Brandt (previous) (diff)

comment:5 Changed 2 years ago by Frank Löffler

Just a note that the proposed branch seems to work well also with the Parma-thorns (compilation runs through and no new tests fail compared to master branch).

comment:6 Changed 20 months ago by Roland Haas

To be included in the flesh it should ideally adhere to the flesh coding standards, see #1520 .

comment:7 Changed 7 months ago by Roland Haas

Piraha is now used but does not adhere to the flesh coding standards.

comment:8 Changed 4 months ago by Roland Haas

Piraha's source code should be cleaned up so that it adheres to the flesh coding standards.

comment:9 Changed 4 months ago by Steven R. Brandt

I'd thought I'd done this. Can you point me to the coding standards you want me to adhere to? Thanks.

comment:10 Changed 4 months ago by Roland Haas

Please see: https://bitbucket.org/cactuscode/cactus/raw/387fae8c18ff04c3368f038d708b801b94e1ca2f/doc/MaintGuide.pdf section A2 in particular the grdoc headers and the indentation rules.

comment:11 Changed 4 months ago by anonymous

Also please see the comments in #1520 which points to the online version of the maintainers guide as well as naming conventions for internal functions.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as accepted The owner will remain Steven R. Brandt.
Next status will be 'review'.
as The resolution will be set.
to The owner will be changed from Steven R. Brandt to the specified user.
The owner will be changed from Steven R. Brandt to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.