Should 'reply_markup' field exist in simple text message ?

[kolyande]

Hello, and Thanks to all contributors!
I found that, the default value for argument 'replyMarkup' in method Api::sendMessage

tgbot-cpp/include/tgbot/Api.h

Line 183 in d2bf07f

GenericReply::Ptr replyMarkup = std::make_shared<GenericReply>(),

, the way it is checked inside method's body

tgbot-cpp/src/Api.cpp

Line 150 in d2bf07f

if (replyMarkup) {

and the way it is parsed then in method TgTypeParser::parseGenericReply - ALL leads to existing of 'reply_markup' field with empty value "" for simple text messages like this: {"chat_id":"1","text":"Hello","reply_markup":""} .
Is it right behaviour ? Does it follow the Telegram bot API doc https://core.telegram.org/bots/api#message where this field is described as Optional ?

[llnulldisk]

Thank you for the message. No, that is not the correct behaviour. The empty field is just unnecessary. I'm surprised I didn't see that before either. I created a pull request that should fix it.