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

Open
veripoolbot opened this issue Apr 14, 2017 · 2 comments
Open

Auto-alignment on comments in PORT declaration #1157

veripoolbot opened this issue Apr 14, 2017 · 2 comments
Labels

Comments

@veripoolbot
Copy link
Collaborator

@veripoolbot veripoolbot commented Apr 14, 2017


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

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Apr 14, 2017


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

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Apr 26, 2017


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
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.