Support Request #3713
closedEBPRODUCTSUPPORT-1057 ADTF File Lib and Unicode
Description
Supportanfrage
Hallo Florian,
Wir haben festgestellt das die ADTFFilelib kein Unicde unterstützt.
Folgendes müsste wohl mindestens angepasst werden:
OLTT's ADTF File Reader plugin depends on adtf_file library, specifically - Reader class, which takes std::string as filename. To fix just this, the following is needed (bare minimum):
1. adtf_util library: a separate constructor for Reader is needed that accepts std::wstring
2. a_util library: Path class (needs to be updated to take std::wstring (and wchar_t*) for path
3. adtf_util library: cFile class needs to be updated to call CreateFileW (Win32 API function) instead of CreateFileA which it currently calls
4. adtf_util library: get_sector_size_for function needs updating so that it makes call to GetDiskFreeSpaceW (Win32 API function)
there also seem to be dependencies into DDL library: cDDLImporter wants to SetFile for .description file using original filename, which will want to load XML file (cDOM class - method of interest is ::load but whole class needs to be checked too as it uses Pugi and passes its arguments to it - fortunately, Pugi has load method for both narrow and wide string filenames).
Was denkst du darüber?
Wie ist dann der Änderungsprozess? Wir machen einen Branch mit unseren Änderungswünschen auf und dann? Wann erfolgt der Review bzw. die Entscheidung wann die Änderung und welche übernommen wird?
Best regards,
Anja Winkler
Lösung
Der Code kommt als Git Clone, ihr oder eure Kunden können gern Verbesserungsvorschläge umsetzen und die Diffs zurückspielen.
Die bisherigen Anforderungen an die Library sind überschaubar bzw. leiten sich aus ADTF ab.
Link zum adtf_file library Repo
Für das Integrieren von Änderungen von außen gibt's bei uns noch keinen Workflow. Ich denke das werden wir aber sehr unkompliziert handhaben, am besten über solche Support Tickets. entwicklen, und dann ein Ticket mit einem angehängten Patch erstellen und wir schauen uns das zeitnah an. Dabei bitte auch immer die Abwärtskompatibilität (sowohl binär als auch source) im Auge behalten
Updated by hidden over 5 years ago
- Project changed from Public Support to 7
- Status changed from New to In Progress
- Topic set to FileLibrary::Reader
- Customer set to ELEKTROBIT
- Department set to SUPPORT
- Affected Products ADTF File Library 0.2.0 (BETA) added
Hallo Anja,
der Code kommt als Git Clone, ihr oder eure Kunden können gern Verbesserungsvorschläge umsetzen und die Diffs zurückspielen.
Die bisherigen Anforderungen an die Library sind überschaubar bzw. leiten sich aus ADTF ab.
@Martin: Kannst du das Thema bitte anschauen und monitoren ?
Updated by hidden over 5 years ago
Hallo zurück,
habt ihr da einen Link zum Repo für uns?
Danke und Grüße,
Anja
Updated by hidden over 5 years ago
- Status changed from In Progress to To Be Closed
- Resolution set to Solved Issue
- Platform Windows 7 64bit added
Updated by hidden over 5 years ago
Hallo Martin,
wie sieht es mit der DDL und der autil library aus? Gibt es hier auch Git Repos?
Wie ist dann der Änderungsprozess? Wir machen einen Branch mit unseren Änderungswünschen auf und dann? Wann erfolgt der Review bzw. die Entscheidung wann die Änderung und welche übernommen wird?
Danke und Grüße,
Anja
Updated by hidden over 5 years ago
- Status changed from To Be Closed to In Progress
Hallo Martin,
es gab hierzu noch eine Rückfrage
Updated by hidden over 5 years ago
Hallo Anja,
habt ihr auf den Link geklickt? Da findet ihr auch die a_util und ddl.
Für das Integrieren von Änderungen von außen gibt's bei uns noch keinen Workflow. Ich denke das werden wir aber sehr unkompliziert handhaben, am besten über solche Support Tickets. Also entwicklen, und dann ein Ticket mit einem angehängten Patch erstellen und wir schauen uns das zeitnah an. Dabei bitte auch immer die Abwärtskompatibilität (sowohl binär als auch source) im Auge behalten.
Grüße,
Martin
Updated by hidden over 5 years ago
- Status changed from In Progress to Customer Feedback Required
Updated by hidden over 5 years ago
Hallo Martin,
ok, vielen Dank für die Info. Du kannst das Ticket schließen.
Best regards,
Anja Winkler
Updated by hidden over 5 years ago
- Subject changed from EBPRODUCTSUPPORT-1057 ADTF File Lib und Unicode to EBPRODUCTSUPPORT-1057 ADTF File Lib and Unicode
- Description updated (diff)
- Status changed from Customer Feedback Required to To Be Closed
Updated by hidden over 5 years ago
- Status changed from To Be Closed to Closed
- Private changed from Yes to No