Unneeded uniq()

This is apparently a minor issue, but I just noticed and decided to report it:

https://github.com/mattn/vim-lsp-settings/blob/7166674dc49306a6fcaff19d4dde8bca95271460/autoload/lsp_settings/profile.vim#L101-L110

First you create a unique list of commands, then pass it to uniq() which makes no difference. Unless I'm missing something.

On a side note I thought if it could be done in a more compact way, and here's what I ended up with:

let Flatten = {d -> reduce(values(d), {r, i -> extend(r, i)})}
let l:servers = mapnew(l:settings, 'mapnew(v:val, "v:val.command")')
let l:servers = uniq(sort(Flatten(l:servers)))

Not that it's necessarily better.

Open autoload/lsp_settings/profile.vim at lines 101-110 and inspect how the command list is made unique before uniq() is called. Confirm whether the earlier operation already guarantees uniqueness, then remove only the redundant step if appropriate and verify that the generated server command list is unchanged.