Opened 5 years ago
Last modified 5 years ago
#8587 open defect
Background color is black instead of default background color
Reported by: | Indiana Kernick | Owned by: | |
---|---|---|---|
Priority: | minor | Component: | avutil |
Version: | git-master | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Reproduced by developer: | no | |
Analyzed by developer: | no |
Description
The following snippet has a black background color. It stands out as wrong in the output of the ffmpeg command-line tool.
[libx264 @ 0x7fddb3011a00]
All other text in the output has the default background color but this snippet has a black background color. The background color of the terminal is not necessarily black. It might be gray or some other color.
To be clear:
Black background "\x1B[40m" Default background "\x1B[49m"
"\x1B[49m"
should be used instead of "\x1B[40m"
.
I believe that the snippet is produced whenever the H.264 codec is used. The snippet also has a cyan foreground color so it stands out as the only colored text in the log. I don't have a problem with the foreground color but as I've already said, the background color stands out as very wrong.
Upon checking the version I was using, I found another instance of this issue.
ffmpeg --version
I realise that I should use a single dash but the error message that I got had a black background color.
Unrecognized option '-version'. Error splitting the argument list: Option not found
An even shorter way to reproduce this issue is invoke ffmpeg without an arguments
ffmpeg
The message at the end of the log has a black background color.
Use -h to get full help or, even better, run 'man ffmpeg'
To summarise, there are some instances where "\x1B[40m"
is being used when "\x1B[49m"
would be more appropriate.
Change History (14)
follow-up: 2 comment:1 by , 5 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
comment:2 by , 5 years ago
comment:4 by , 5 years ago
Replying to richardpl:
--options are never supported use -option instead.
The issue is not about that at all.
comment:5 by , 5 years ago
Resolution: | invalid |
---|---|
Status: | closed → reopened |
Why was this closed? I'm not asking why I received an error message after typing ffmpeg --version
as richardpl seems to think. The report is about the formatting of the error message itself. Balling has reproduced the issue and shown a screenshot.
comment:6 by , 5 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
comment:7 by , 5 years ago
Can you please attach screenshot of a master version and with normal h264 reencode and normal terminal, gnome-terminal.
comment:8 by , 5 years ago
Please avoid adding more screenshots, all interested parties can reproduce themselves.
comment:9 by , 5 years ago
Version: | 4.2 → git-master |
---|
follow-up: 12 comment:10 by , 5 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Patch sent, this is possibly wont-fix.
comment:11 by , 5 years ago
Component: | ffmpeg → avutil |
---|
follow-up: 13 comment:12 by , 5 years ago
Status: | reopened → open |
---|
Replying to cehoyos:
Patch sent, this is possibly wont-fix.
The obvious problem here is that you do not close colour and background colour on exit(), this affects new line after exit, that is bad. I wanted to report it long ago, BTW, did not find the power to do so((
Please avoid adding more screenshots
It will not affect you server, everything is on imgur.
comment:13 by , 5 years ago
Replying to Balling:
Replying to cehoyos:
Patch sent, this is possibly wont-fix.
The obvious problem here is that you do not close colour and background colour on exit(), this affects new line after exit, that is bad. I wanted to report it long ago, BTW, did not find the power to do so((
This is not what was reported here and I am not completely convinced that there is an issue that can be fixed in FFmpeg.
Please avoid adding more screenshots
It will not affect you server, everything is on imgur.
You misunderstand:
This is not about the possible effect on a server but the fact that screenshots should be avoided on this bug tracker at (nearly) all costs, this ticket is a rare exception (but I don’t consider the attached screenshots particularly useful).
comment:14 by , 5 years ago
This is not what was reported here and I am not completely convinced that there is an issue that can be fixed in FFmpeg.
Well, the background is a preference issue but the fact you leak colour to new line is strange ;) And this also happens inside the output...
Replying to richardpl:
Well, there is a also this issue:
Did not check master though.