Attention is currently required from: pespin.

View Change

2 comments:

• File src/pdch.cpp:

  • Patch Set #2, Line 1316: ts == pdch

    Same here: comparing foo_ts to pdch may look confusing, so it's better to have a unified naming by doing s/_ts/_pdch/.

• File tests/alloc/AllocTest.cpp:

  • Patch Set #2, Line 186: tbf->control_ts

    Maybe move NULL checking to pdch_name()? This way one could pass PDCH pointers directly without worrying if it's NULL.

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