[Bug 753613] mssdemux: PlayReady WRM parsing support

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=753613

--- Comment #8 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Well, the {} stripping is not elegant, but that's string ops in C for you. I
don't think using a regex will improve it much. You could also do something
like, i.e. strip in place. If that's less code or nicer, I don't know. It
doesn't really matter much imho.

  gsize id_len = strlen (system_id);

  if (system_id[id_len - 1] == '}') {
    system_id[id_len - 1] = '\0';
    --id_len;
  }

  /* move string one forward including the terminator */
  if (system_id[0] == '{')
    memmove (system_id, system_id + 1, id_len - 1 + 1);

Is the patch otherwise ready for review or is it still work in progress?
("started a patch" sounds like work in progress)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.