#media-maint 2021-02-12,Fri

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)

WhoWhatWhen
sailushverkuil: Huomenta! [10:28]
.... (idle for 16mn)
hverkuilsailus: hi [10:44]
....... (idle for 31mn)
sailushverkuil: How do you do? [11:15]
hverkuilfine, you as well? [11:17]
sailusFine, too!
On the %p4cc patchset --- would you prefer fourcc codes stripped of the spaces?
(I think I would, at least at the end.)
[11:18]
hverkuilI would actually prefer to keep the spaces.
Certainly if they appear at the beginning or in the middle.
[11:20]
sailusWe don't have any of such codes. [11:21]
hverkuilSo if you keep those (and I think you have to), then it is inconsistent to strip trailing spaces. [11:21]
sailusBut fair enough, Petr was asking the same.
I'll change it to keep them then.
[11:21]
hverkuilIt's a four character code, just because one character is a space, doesn't mean that it isn't important. It's still a character. [11:21]
sailusHow about unprintable characters?
Their hexadecimal numeric value is printed currently, but I guess we could do with something else.
[11:22]
hverkuilToday the only non-printable character can be the first byte if bit 7 is set to indicate big-endian, and that's already handled. [11:23]
sailusPetr thought it wasn't clear in which format the number would be, and I think DRM used a question mark in such a case.
Yes.
So this is a hypothetical case.
[11:23]
hverkuilYes. It can be replaced by ? if you want to make it really robust, but it shouldn't happen.
Perhaps for a formatter like this being robust is not such a bad idea :-)
[11:23]
sailusI'm a bit hesitant of using characters the shell could expand.
It's unlikely to be a problem, but there are choices that certainly aren't.
[11:24]
hverkuilOr replace with . [11:24]
sailusSuch as .,
Yes.
[11:24]
hverkuilor _ [11:24]
sailusI'll propose . [11:25]
hverkuilJust add a comment that unprintable characters shouldn't happen for a valid fourcc. [11:25]
sailusYes.
I'll send a new version soon then.
[11:26]

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)