Polyspace: expression must have a constant value

6 visualizzazioni (ultimi 30 giorni)
Hello guys,
I have this C++17 code:
#define I2C_BUS_MAX_BUS_ITEMS 3
struct my_bus_t {
int bus{-1};
bool is_external;
};
static inline constexpr my_bus_t initBus(int bus)
{
my_bus_t ret{};
ret.bus = bus;
ret.is_external = false;
return ret;
}
constexpr my_bus_t buses[I2C_BUS_MAX_BUS_ITEMS] = {
initBus(1),
initBus(2),
};
This is the configuration setting.
It compiles well, but there is a problem in Polyspace: "expression must have a constant value".
Thank you in advance.
  2 Commenti
Walter Roberson
Walter Roberson il 28 Apr 2022
Which expression is it flagging?
Zulham Mr
Zulham Mr il 28 Apr 2022
constexpr my_bus_t buses[I2C_BUS_MAX_BUS_ITEMS] = {
initBus(1),
initBus(2),
};
This one.

Accedi per commentare.

Risposta accettata

Anirban
Anirban il 29 Apr 2022
Modificato: Anirban il 29 Apr 2022
You are initializing a three-element constexpr array with only two elements. If you change the line:
constexpr my_bus_t buses[I2C_BUS_MAX_BUS_ITEMS] = {
initBus(1),
initBus(2),
};
To something like:
constexpr my_bus_t buses[I2C_BUS_MAX_BUS_ITEMS] = {
initBus(1),
initBus(2),
initBus(2)
};
The error will go away. I am not sure why your compiler did not flag this.
  3 Commenti
Anirban
Anirban il 2 Mag 2022
The C++ reference for constexpr states that constexpr variables must be immediately initialized.
In this case, you have partially initialized the array. This is not an error by itself, and my guess is, that is why the compiler does not complain. If you try to initialize the uninitialized elements later, the compiler should show an error.
Indeed, Polyspace is being stricter than your compiler. But, as you can see, partially initializing a constexpr array is an error because it cannot be initialized fully later.
Zulham Mr
Zulham Mr il 3 Mag 2022
Thank you for your explanation.

Accedi per commentare.

Più risposte (0)

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by