Changeset View
Changeset View
Standalone View
Standalone View
src/plugins/marlinplugin.h
Show First 20 Lines • Show All 50 Lines • ▼ Show 20 Line(s) | 40 | public: | |||
---|---|---|---|---|---|
51 | 51 | | |||
52 | /** | 52 | /** | ||
53 | * @brief Return Plugin name | 53 | * @brief Return Plugin name | ||
54 | * @return Marlin | 54 | * @return Marlin | ||
55 | */ | 55 | */ | ||
56 | QString name() const override; | 56 | QString name() const override; | ||
57 | 57 | | |||
58 | /** | 58 | /** | ||
59 | * @brief Translate common commands to firmware specific command. | ||||
60 | * @param command: command to translate | ||||
61 | * @return firmware specific translated command | ||||
62 | */ | ||||
63 | QByteArray translate(const QString &command) override; | ||||
64 | | ||||
65 | /** | ||||
59 | * @brief validateCommand to filter commands from messages | 66 | * @brief validateCommand to filter commands from messages | ||
60 | * @param lastMessage: last Message from printer | 67 | * @param lastMessage: last Message from printer | ||
61 | */ | 68 | */ | ||
62 | void validateCommand(const QString &lastMessage) override; | 69 | void validateCommand(const QString &lastMessage) override; | ||
70 | | ||||
71 | private: | ||||
72 | /** | ||||
73 | * @brief Convert a filename to 8.3 Because Marlin will only accept those, Generates a 8.3 like Filename. | ||||
74 | * This is Close to but not a 8.3 filename for instance we do not check the Filesystem so ~# will not be added if files with similar names are on the sd card | ||||
75 | * Marlin will overwrite files with the samename. | ||||
76 | * @param fileName: to convert to 8.3 format. | ||||
77 | * @return Return a 8.3 Format name | ||||
78 | */ | ||||
79 | QString to83(const QString &fileName); | ||||
patrickelectric: This should be something shared in all plugins, it's not marlin specific.
Also to83 is not a… | |||||
As soon as i see a plugin other then marlin that needs this function. We only have sd support on marlin and repetier so far and repetier uses full names rizzitello: >This should be something shared in all plugins, it's not marlin specific.
As soon as i see a… | |||||
63 | }; | 80 | }; |
This should be something shared in all plugins, it's not marlin specific.
Also to83 is not a good name, maybe shortFilename