limit indent size to 8 - #5
Conversation
addresses timakro#4
|
I think the intent was to fall back to user preferences or file-type defaults in the case of a unrealistically large indentation? For example, given a file like this curl https://example.com/ -H "Content-Type: application/json" \
-H "Accept: application/json"you wouldn't want to set shiftwidth and and softtabstop to 8, right? You probably had something more like de2fd6c in mind. |
|
Yes, such an example. User preferences are fine as well. I was not sure whether you preferred a max value instead. |
|
I don't mean to make the maximum indentation value configurable, sorry for the confusion. I think 8 is good. I was trying to say that your PR would change shiftwidth and softtabstop given my example file while the intent as I understand it is to not change those settings in this scenario. |
|
Yes, counting only up to 8 is a better solution, thank you. Edit: indent is computed relative to the last indent |
addresses #4