search

denniskempin / align-arguments

Sublime Text 3 Plugin to automatically format and align argument lists on the opening bracket
MIT License
6 stars 3 forks source link

Doesn't handle tabs #2

Open chuckfossen opened 4 years ago

chuckfossen commented 4 years ago

If I use leading tabs, this plugin does not indent correctly. For example:

    result = my_function(param1, param2, param3, param4, para5, param6, param7,
param8, param9, param10, param11, param12, param13);

turns into:

    result = my_function(param1, param2, param3, param4, para5, param6, param7,
                      param8, param9, param10, param11, param12, param13);

Also, the indentation should use leading tabs as well.

denniskempin commented 4 years ago

I had not considered mixing tabs and spaces as that's a quite uncommon style.

Frankly I have abandoned this plugin. I haven't been using Sublime in years, and it's also gotten somewhat unnecessary as it's quite common these days to just run all source through a formatter.

That said, I'd be happy to accept a pull request and will push it to packagecontrol.io, or if you'd like, I can see if I can transfer the package rights to you.

chuckfossen commented 4 years ago

Leading tabs is the standard for linux kernel source code. I'll take a look at the changes necessary and alternatives to see if I should take this on. Thanks for your response.