Support for Latin1 (fixed)

[jesteves]

Hello. EditorConfig supports charset Latin1 with setting charset = latin1 but Atom (1.30.0) does not recognize it like so. It has to be set as ISO 8859-1 (or Western (Windows 1252), actually).

I've implemented this fix:

diff -C5 orig/editorconfig/index.js .atom/packages/editorconfig/index.js
*** orig/editorconfig/index.js
--- .atom/packages/editorconfig/index.js
***************
*** 221,230 ****
--- 221,235 ----

  		settings.charset = ('charset' in config) ?
  			config.charset.replace(/-/g, '').toLowerCase() :
  			'auto';

+ 		if (settings.charset === 'latin1') {
+ 			/* Atom knows nothing about Latin1... */
+ 			settings.charset = 'iso88591';
+ 		}
+
  		ecfg.applySettings();
  	}).catch(Error, e => {
  		console.warn(`atom-editorconfig: ${e}`);
  	});
  }

Hope you can consider it for merge.

Regards!

PS & disclaimer: ⚠️ I tried forking the project to make a pull request but I just found that v2.2.2 on GitHub is not the same v2.2.2 on atom.io. That is, the fork does not match what can be downloaded for installation...

Involved .editorconfig-files

root = true

[*]
indent_style = space
indent_size = 2
end_of_line = lf
trim_trailing_whitespace = true
insert_final_newline = true
charset = latin1

Directory structure

Skipped.

Installed packages

Built-in Atom Packages (92)
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
└── [email protected]

Community Packages (19) /Users/julian/.atom/packages
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
└── [email protected]

[the-j0k3r]

Why not submit a pull request instead?

[Alhadis]

Added, thanks! I also considered latin-1 or latin 1 (matched case-insensitively) as possible aliases too. 👍