use array index in identifiers

SystemRDL / PeakRDL-Markdown

Markdown exporter for the PeakRDL toolchain

GNU General Public License v3.0

4 stars 7 forks source link

use array index in identifiers #19

Closed bwhitchurch closed 5 months ago

bwhitchurch commented 5 months ago

This changes the identifier string for register arrays to use the array index rather than size of the array dimension. It addresses the basic issue raised in #5.

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9355208716

Details

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.095%

Totals
Change from base Build 8711348166:	0.0%
Covered Lines:	148
Relevant Lines:	168

💛 - Coveralls

jeras commented 5 months ago

I just tested it, it works as described.

Array elements in the address table are now properly indexed. It works for multidimensional arrays too.

The repetitive documentation for each array element remains. All instances have the same heading, giving no indication which index are they, they differ only in the absolute address.