Revert "Revert Rename of dlt_common. (#512)"

[LocutusOfBorg]

This reverts commit a3ad76c19cea1a52a8870d435e6cb4f1de4d2621.

[LocutusOfBorg]

reopened

[alexmucde]

Thanks, i reverted already with this PR #512

[LocutusOfBorg]

So, can you please explain what broke? We should really convert it into a c++ file, if containing c++ code, right?

[alexmucde]

@LocutusOfBorg Sorry i have not much tiem at the moment. The Windows build was not working, thats why i have reverted it. If you can also have a look into Windows would be nice.

[LocutusOfBorg]

@alexmucde does the pipeline have some link? I don't have any windows machine, nor I know how to build, but if the pipeline runs on GitHub I can try to fix...

[LocutusOfBorg]

@alexmucde can you please try this solution instead? It fixes the build for me

[alexmucde]

@LocutusOfBorg I get the following error:

Bibliothek "..\release\qdlt.lib" und Objekt "..\release\qdlt.exp" werden erstellt. qdltoptmanager.obj : error LNK2019: Verweis auf nicht aufgel”stes externes Symbol "__imp_GetWindowThreadProcessId" in Funktion ""public: void __cdecl QDltOptManager::parse(class QStringList *)" (?parse@QDltOptManager@@QEAAXPEAVQStringList@@@Z)". ..\release\qdlt.dll : fatal error LNK1120: 1 nicht aufgel”ste Externe

[alexmucde]

The following line is missing in qdlt.pro:

win32:LIBS += User32.lib

This is already integrated in the master.

[LocutusOfBorg]

is it ok now? I did sync master

[alexmucde]

@LocutusOfBorg Sorry, we have some conflicts now.

[LocutusOfBorg]

Rebased, I admit after https://github.com/COVESA/dlt-viewer/pull/594 this is not useful anymore, but we can still simplify somewhere the speed_t definition in one single place, to have better code and more clean

[LocutusOfBorg]

thanks!