Created attachment 34673 [details]
I'm attaching a slightly updated symbols/fi with the following changes:
- added header about the file itself as required by the standard SFS 5966, clean up old obsoleted comments
- (scripted) whitespace clean up
- added missing xkb_symbols header for the smi variant
- corrected name for Eliminate dead keys variant (it is a variant of Classic map not the default variant)
- added few sometimes used symbols from the default variant to Classic variant
- reorganized Classic variant to follow the default variant grouping
- no modifications to the default variant
Due to whitespace and line ordering changes a patch would be a bit hard to read so the new file is attached. If you want to inspect the changes in detail I suggest using 'diff -uw'.
I'm CC'ing Troy with whom I've already discussed about this update (and last time we talked he was ok with it).
Sorry, the difference between current version in git and your file is large enough. Could you please submit a patch instead - making sure that all changes are necessary?
Indeed, there were quite a few changes for a one go. I'll attach incremental patches that'll make it more clear what's going on. Please ask for clarifications if needed.
Created attachment 35630 [details]
clean up header
This patch just cleans up the file header.
Created attachment 35631 [details]
add std header
This patch adds the standard mandated definition of the default keymap. Add/remove some blank lines for readability, adjust keymap header to match other headers.
Created attachment 35632 [details]
This patch adds the missing keymap header to the smi variant, corrects some comments and changes Ooblique->Oslash for consistency.
Created attachment 35633 [details]
add new keys to classic variant
This patch adds few new keys to the classic variant where feasible to make some features of the default keymap available also on the classic one. Adds missing compose include, uses include directive to map nbsp.
Created attachment 35634 [details]
This patch syncs the nodeadkeys variant after the previous changes.
Created attachment 35635 [details]
This patch just sorts the key mappings in the classic map to be follow the ordering the default keymap. No changes to mappings.
Created attachment 35636 [details]
Scripted whitespace clean up, zero changes to anything else which can be verified by running diff -w for the original and patched maps.
Thank you very much! Good timing, just before the release.
The patches are fine and easy to understand. I am just a bit surprised though they are not formatted as git commits.
Anyway, committed. Good that you settled that with Troy