Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't reject 8.3-filenames with chars > 0x7f #3648

Merged
merged 1 commit into from
May 1, 2016

Conversation

Blue-Marlin
Copy link
Contributor

@Blue-Marlin Blue-Marlin commented Apr 30, 2016

Don't reject 8.3-filenames with chars > 0x7f

Windows produces 8.3-filenames with chars > 0x7f. Those have been rejected
by Marlin until now.

With these 'malformed' filenames can now be worked:

  • In the LCD menu. If Marlin has not the same charset as the used OS, when creating the file, the 'strange' chars are displyed wrong - but you can work with.
  • With RepetierHost (V1.6.1 tested) - full support. Characters are
    displayed as '?'

  • With Octoprint (1.2.10 tested) the files do not appear in the files
    area. At the console, listed with M20 they appear with a '�'.
  • With Pronterface the files appear in the sd-window but you can't start
    them. They are mangled by Pronterface. The names are altered and than
    rejected by Marlin. In the console they appear with different but not the
    correct characters.

All in all, a little step forward.

Fix for #3593

Don't mangel 8.3-filenames with chars > 0x7f

Windows produces 8.3filenames wit chars > 0x7f. Those have been rejected
by Marlin until now.

With these 'malformed' filenames can now be worked with:
In the LCD menue
With RepetierHost (V1.6.1 tested) - full support. Characters are
displayed as '?'
With Octoprint (1.2.10 tested) the files do not appear in the files
area. At the console, listed with M20 they appear with a '�'.
With Pronterface the files appear in the sd-window but you can't start
them. They are mangled by pronterface. The names are altered and than
recected by Marlin. In the console they apper with differen but not the
correct characters.

All in all a little step forward.

Fix for MarlinFirmware#3593
@thinkyhead thinkyhead merged commit 77d6e52 into MarlinFirmware:RCBugFix May 1, 2016
Blue-Marlin added a commit to Blue-Marlin/Makaira that referenced this pull request May 2, 2016
Fix for #6 3593, MarlinFirmware#3648
this time excludung 'DEL' instead of accidently '~'.
@Blue-Marlin Blue-Marlin deleted the 83filenames branch May 2, 2016 14:13
nomukaiki pushed a commit to nomukaiki/Marlin that referenced this pull request May 4, 2016
Fix for MarlinFirmware#6 3593, MarlinFirmware#3648
this time excludung 'DEL' instead of accidently '~'.
@jbrazio jbrazio modified the milestone: 1.1.0 Jul 18, 2016
CONSULitAS pushed a commit to CONSULitAS/Marlin-K8200 that referenced this pull request Aug 18, 2016
Fix for MarlinFirmware#6 3593, MarlinFirmware#3648
this time excludung 'DEL' instead of accidently '~'.
drewmoseley pushed a commit to drewmoseley/Marlin that referenced this pull request Nov 8, 2023
cleanup: Expand tabs to fixup suspicious indentation warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants