Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto-alignment on comments in PORT declaration #1157

Closed
veripoolbot opened this issue Apr 14, 2017 · 2 comments · Fixed by #1775
Closed

Auto-alignment on comments in PORT declaration #1157

veripoolbot opened this issue Apr 14, 2017 · 2 comments · Fixed by #1775

Comments

@veripoolbot
Copy link
Collaborator


Author Name: CHOON LIN LEE
Original Redmine Issue: 1157 from https://www.veripool.org


Original Code

module top
(
//Inputs
input CLK, //System Clock
input RST, //Active High Reset
input [3:0] CONTROL, //Control decoder.

//Outputs
output reg [7:0] LIVE_DATA, //1 byte data
output VALID //Valid Flag
);

Is there any feature to auto-align the comments? After auto-align comments, it should be as below:
module top
(
//Inputs
input CLK, //System Clock
input RST, //Active High Reset
input [3:0] CONTROL, //Control decoder.

//Outputs
output reg [7:0] LIVE_DATA,//1 byte data
output VALID //Valid Flag
);

Having issues to align those comments especially the source is from other text editor. EMACS Verilog mode will mis-align the comments for port declaration part.

@veripoolbot
Copy link
Collaborator Author


Original Redmine Comment
Author Name: CHOON LIN LEE
Original Date: 2017-04-14T05:57:22Z


After auto align the comments, it should be like below:

@veripoolbot
Copy link
Collaborator Author


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2017-04-26T16:58:48Z


I can see the usefulness. If you want to make a patch we'll consider it, otherwise note there is no present active development of new indentation features, so you're in for a long wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant