search

richjyoung / vscode-modern-vhdl

Modern VSCode VHDL Support
MIT License
30 stars 12 forks source link

Maximum line length setting #8

Closed wimille closed 5 years ago

wimille commented 5 years ago

Hi

The following problem occurs when i use the stutter comments mode: As my code is indented, i start to write my code at column 2 (two spaces characters at the begining of my line). But when i wrote '---', the VHDL extension create a comment '-----' line. But instead of ends at line 110 (which is configured as line length) the extension wrote 110 charcters '---'.

It is possible to take the start column and write only characters to reach line lenght ?

Thanks. Best regards

richjyoung commented 5 years ago

I will add an option to either treat the setting as a constant width to insert, or a maximum line length to insert up to.

Do you know if this behaviour is different to the Emacs behaviour this is based on? I would like to set the default behaviour to mirror that of Emacs if possible.

Thanks for reporting this.

On Mon, 8 Apr 2019, 13:57 wimille, notifications@github.com wrote:

Hi

The following problem occurs when i use the stutter comments mode: As my code is indented, i start to write my code at column 2 (two spaces characters at the begining of my line). But when i wrote '---', the VHDL extension create a comment '-----' line. But instead of ends at line 110 (which is configured as line length) the extension wrote 110 charcters '---'.

It is possible to take the start column and write only characters to reach line lenght ?

Thanks. Best regards

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/richjyoung/vscode-modern-vhdl/issues/8, or mute the thread https://github.com/notifications/unsubscribe-auth/AD3aKcOy0eDOHvUJEfH8siEMOr4r65HXks5vezy9gaJpZM4ciHZq .

wimille commented 5 years ago

Hi, Sorry i don't know the behavior of Emacs.

richjyoung commented 5 years ago

I've checked the settings description and the behaviour is correct, the length setting is the width to be inserted, not a maximum.

I agree that you should be able to specify that as well, so will implement that as a new feature.