Attention is currently required from: jolly.

View Change

2 comments:

• File src/host/layer23/src/mobile/gsm48_mm.c:

  • Patch Set #5, Line 1427: uint32_t ref, uint16_t class)

    using "class" as param name is asking for trouble eg. when compiling with a C++ compiler due to being a reserved keyword. It is even rendered differently in here in gerrit. Maybe call it "klass".

  • Patch Set #5, Line 1449: LOGP(DMM, LOGL_ERROR, "Invalid message class 0x%x. Please fix!", class);

    0x02%x?

To view, visit change 34478. To unsubscribe, or for help writing mail filters, visit settings.