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

Better support for package_import_declaration in the module_ansi_header #9

Open
pdonahue-ventana opened this issue Aug 14, 2019 · 0 comments

Comments

@pdonahue-ventana
Copy link

I'm very excited to find this formatter and it seems very promising. I do have a concern that I noticed right away.

I have this input file which uses a non-empty package_import_declaration in the module_ansi_header:

module Foo
  import FooPkg::*;
  import BarPkg::*;
  import GlobalPkg::*;
  import OtherPkg::*;
  (
   input logic  clk;
   output logic something;
  );

endmodule : Foo

The output is:

module Foo import FooPkg::*; import BarPkg::*; import GlobalPkg::*; import OtherPkg::*; (input logic clk; output logic something;);
    
    endmodule : Foo

It seems more reasonable to have each import on a separate line.

Thanks.

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

No branches or pull requests

1 participant