Johannes Stezenbach wrote: > It still would be hard to read. How about: > > p_program_descriptor->program_number = > ((uint16_t) buf[pos] << 8) | buf[pos + 1]; > I was thinking about cleaning up a bit of code in dst_ca.c, ca_set_pmt() from where all this commotion is coming up .. Any suggestions on the cleanups ? Thanks, Manu