- Added explicit type casting on:

* Writing data bytes, IDE, DLC and MsgId on pack step

* Explicit unsigned nature ('U') to message CANID macro value

- ```-Wpedantic``` doesn't complain anymore on generated code (tested on test.dbc only)

- Added 'define' with value for filling initial value in frame's bytes before packing signals.

The value can be set in the user scope configuration file (driver-config)

- Added enhanced DLC handling:

* Added max detected DLC value (as a macro in main driver)

* User can specify its own max DLC value if necessary (by setting corresponding macro in driver-config file)

* Functions use new 'VALIDATE_DLC' test when checks the limit of frame data bytes count

- Include files in quotes instead of angle brackets

- No more struct, pack and unpack for frames with no signals (empty frames). IDs and frame related info is left

- Fixed an issue in 'findversion' function

[Unreleased]: https://github.com/astand/c-coderdbc/compare/v3.0...HEAD

[v3.0]: https://github.com/astand/c-coderdbc/compare/v2.9...v3.0

// save max dlc value from message list for printing on the following generation steps

val_maxDlcValueFromDbcList = dlist.maxDlcValue;

fwriter.Append("#include \"%s-config.h\"", fdesc->gen.drvname.c_str());

"#include \"canmonitorutil.h\"\n"

// set macro name for max dlc value based on driver name

std::string maxDlcMacroName = fdesc->gen.DRVNAME + "_MAX_DLC_VALUE";

// set macro name for dlc validation

prt_dlcValidateMacroName = fdesc->gen.DRVNAME + "_VALIDATE_DLC";

// set macro name for initial data byte value based on driver name

prt_initialDataByteValueName = fdesc->gen.DRVNAME + "_INITIAL_BYTE_VALUE";

// print part with max DLC macro

fwriter.Append("// DLC maximum value which is used as the limit for frame's data buffer size.");

fwriter.Append("// Client can set its own value (not sure why) in driver-config");

fwriter.Append("// or can test it on some limit specified by application");

fwriter.Append("// e.g.: static_assert(TESTDB_MAX_DLC_VALUE <= APPLICATION_FRAME_DATA_SIZE, \"Max DLC value in the driver is too big\")");

fwriter.Append("#ifndef %s", maxDlcMacroName.c_str());

fwriter.Append("// The value which was found out by generator (real max value)");

fwriter.Append("#define %s %uU", maxDlcMacroName.c_str(), val_maxDlcValueFromDbcList);

fwriter.Append("#endif");

fwriter.Append(1);

// actual macro for final DLC validation in the driver

fwriter.Append("// The limit is used for setting frame's data bytes");

fwriter.Append("#define %s(msgDlc) (((msgDlc) <= (%s)) ? (msgDlc) : (%s))",

prt_dlcValidateMacroName.c_str(), maxDlcMacroName.c_str(), maxDlcMacroName.c_str());

// print initial data byte section

fwriter.Append(1);

fwriter.Append("// Initial byte value to be filles in data bytes of the frame before pack signals");

fwriter.Append("// User can define its own custom value in driver-config file");

fwriter.Append("#ifndef %s", prt_initialDataByteValueName.c_str());

fwriter.Append("#define %s 0U", prt_initialDataByteValueName.c_str());

fwriter.Append("#endif");

fwriter.Append(2);

for (size_t num = 0u; num < sigprt.sigs_expr.size(); num++)

fwriter.Append("#define %s_CANID (%#xU)", m.Name.c_str(), m.MsgID);

if (!m.frameNotEmpty)

{

// do nothing with empty frame, leave only id other relevant constants

fwriter.Append();

continue;

}

if (!m.frameNotEmpty)

{

// do nothing with empty frame

continue;

}

fwriter.Append("#include \"%s-fmon.h\"", fdesc->gen.drvname.c_str());

if (!m.frameNotEmpty)

{

// do nothing, no pack and unpack functions for empty frames

continue;

}

fwriter.Append(" _m->%s = (%s) %s(( %s ), %d);",

sname, PrintType((int)sgs->msg.Signals[num].TypeRo).c_str(),

ext_sig_func_name, expr.c_str(), (int32_t)sgs->msg.Signals[num].LengthBit);

fwriter.Append(" _m->%s = (%s) ( %s );", sname,

PrintType((int)sgs->msg.Signals[num].TypeRo).c_str(), expr.c_str());

fwriter.Append(" _m->%s = (%s) %s_%s_fromS(_m->%s);",

sgs->msg.Signals[num].NameFloat.c_str(),

PrintType((int)sgs->msg.Signals[num].TypePhys).c_str(),

fdesc->gen.DRVNAME.c_str(), sname, sname);

fwriter.Append(" cframe->MsgId = (uint32_t) %s_CANID;", sgs->msg.Name.c_str());

fwriter.Append(" cframe->DLC = (uint8_t) %s_DLC;", sgs->msg.Name.c_str());

fwriter.Append(" cframe->IDE = (uint8_t) %s_IDE;", sgs->msg.Name.c_str());

fwriter.Append(" *_len = (uint8_t) %s_DLC;", sgs->msg.Name.c_str());

fwriter.Append(" *_ide = (uint8_t) %s_IDE;", sgs->msg.Name.c_str());

// this function will print body of packing function

fwriter.Append(" uint8_t i; for (i = 0u; i < %s(%s_DLC); %s[i++] = %s);",

prt_dlcValidateMacroName.c_str(),

sgs->msg.Name.c_str(), arrtxt.c_str(),

prt_initialDataByteValueName.c_str());

fwriter.Append(" _m->%s = (%s) 0;", sgs->msg.CsmSig->Name.c_str(), PrintType((int)sgs->msg.CsmSig->TypeRo).c_str());

fwriter.Append(" _m->%s = (%s) %s_%s_toS(_m->%s);",

sgs->msg.Signals[n].Name.c_str(),

PrintType((int) sgs->msg.Signals[n].TypeRo).c_str(),

fdesc->gen.DRVNAME.c_str(),

fwriter.Append(" %s[%d] |= (uint8_t) ( %s );", arrtxt.c_str(), i, sgs->to_bytes[i].c_str());

fwriter.Append(" %s[%d] |= (uint8_t) ( %s );", arrtxt.c_str(), sgs->msg.CsmByteNum, sgs->msg.CsmToByteExpr.c_str());

// Actual max DLC value from dbc list instance

size_t val_maxDlcValueFromDbcList;

// Macro for default initial frame's data bytes value

std::string prt_initialDataByteValueName;

// Macro for frame DLC validation

std::string prt_dlcValidateMacroName;

PrintType((uint8_t)sig->TypeRo).c_str());

(sig->StartBit + (sig->LengthBit - 1)) : (sig->StartBit));

snprintf(workbuff, WBUFF_LEN, "((_d[%d] >> %dU) & (%s))", bn, bbc - slen, msk[slen].c_str());

snprintf(workbuff, WBUFF_LEN, "((_m->%s & (%s)) << %dU)", sig->Name.c_str(), msk[slen].c_str(), bbc - slen);

snprintf(workbuff, WBUFF_LEN, "(%s(_d[%d] & (%s)) << %dU)", t64.c_str(), bn, msk[bbc].c_str(), slen);

snprintf(workbuff, WBUFF_LEN, "((_m->%s >> %dU) & (%s))", sig->Name.c_str(), slen, msk[bbc].c_str());

snprintf(workbuff, WBUFF_LEN, "(%s(_d[%d] & (%s)) << %dU)", t64.c_str(), bn, msk[8].c_str(), slen);

snprintf(workbuff, WBUFF_LEN, "((_m->%s >> %dU) & (%s))", sig->Name.c_str(), slen, msk[8].c_str());

snprintf(workbuff, WBUFF_LEN, " | ((_d[%d] >> %dU) & (%s))", bn, 8 - slen, msk[slen].c_str());

snprintf(workbuff, WBUFF_LEN, "((_m->%s & (%s)) << %dU)", sig->Name.c_str(), msk[slen].c_str(),