Skip to content
This repository has been archived by the owner on Oct 4, 2020. It is now read-only.

False-Errors With Latin1 #169

Open
tajmone opened this issue Nov 14, 2019 · 1 comment
Open

False-Errors With Latin1 #169

tajmone opened this issue Nov 14, 2019 · 1 comment

Comments

@tajmone
Copy link

tajmone commented Nov 14, 2019

EClint reports as an error the presence of an acute accent ($B4: ´) inside an ISO-8859-1 encoded file configured to charset = latin1:

syntax_test_interpolation-ascii.hug
    11:49 ❌ character out of latin1 range: "�"
@tajmone
Copy link
Author

tajmone commented Nov 14, 2019

Here's a link to the repository where I've encountered this problem:

I had to suppress the charset = latin1 option in the .editorconfig file:

## Hugo Interactive Fiction
###########################
; Hugo sources are strictly in ISO-8859-1 (aka Latin1).
;   https://www.generalcoffee.com/hugo

[hugo/*.{hug,h,g}]
; charset = latin1  # BUGGY: reports acute accent as invalid char.
;                            (see: github.com/jedmao/eclint#169)
indent_style = space
indent_size = unset
end_of_line = unset
trim_trailing_whitespace = true
insert_final_newline = true

because it was reporting the following errors for the hugo/syntax_test_interpolation-ascii.hug file:

$ eclint check syntax_test_interpolation-ascii.hug
syntax_test_interpolation-ascii.hug
    11:49 ❌ character out of latin1 range: "�"
                                                (EditorConfig charset https://goo.gl/hK94EO)
    20:05 ❌ character out of latin1 range: "�"
                                                (EditorConfig charset https://goo.gl/hK94EO)
    20:10 ❌ character out of latin1 range: "�"
                                                (EditorConfig charset https://goo.gl/hK94EO)
    20:15 ❌ character out of latin1 range: "�"
                                                (EditorConfig charset https://goo.gl/hK94EO)
    20:20 ❌ character out of latin1 range: "�"
                                                (EditorConfig charset https://goo.gl/hK94EO)

tajmone added a commit to tajmone/highlight-test-suite that referenced this issue Nov 14, 2019
* Hugo Syntax:
    * Merge `hugo-lang-dev` branch into `master` with new Hugo v1.1.0
      syntax, as submitted to Highlight (See: saalen/highlight!122).
      This should make the Highlight GitLab CI tests for Hugo pass,
      since the Hugo syntax has changed considerably and the old tests
      no longer apply.
* Repository Settings:
    * Tweak and optimize Git setting.
    * Add Hugo patterns to Git and EditorConfig setting.
    * Polish all Hugo files to pass EClint validation.

NOTE: Had to disable `charset = latin1` for Hugo in `.editorconfig`
due to a bug reporting acute accents as an invalid Latin1 chars.
( See: jednano/eclint#169 )
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant