[gst-devel] Re: [gst-cvs] company gst-plugins: gst-plugins/gst/mpegstream/

[Ronald Bultje]

Hey Benjamin,

At 03:42 AM 5/22/03 -0700, you wrote:
>case ranges is a gcc extension, removing them
[..]
>+      } else if (stream_id >= 0xE0 && stream_id < 0xF0) {
>-	case 0xE0 ... 0xEF:
[etc.]

This is... Hideous! Stop this, please. if/else things are
meant for one- or two-order comparisons, not for full lists.
Use multiple case statements instead:

case 0xE0: case 0xE1:
[etc.]
case 0xEE: case 0xEF:
  ..
  break;

That might be a few more lines of code, but it'll actually
look good. multi-order if/else stuff looks *horrible*, imho.

Ronald