[Mesa-dev] [PATCH] fix temperature display sign in hud

Signed-off-by: Benjamin Bellec --- src/gallium/auxiliary/hud/hud_context.c | 5 + src/gallium/drivers/radeon/r600_pipe_common.c | 2 +- src/gallium/include/pipe/p_defines.h | 1 + src/mesa/state_tracker/st_cb_perfmon.c| 5 + 4 files changed, 12

Re: [Mesa-dev] [PATCH] fix temperature display sign in hud

Hi Benjamin, In my opinion, your commit message is a bit confusing I think you meant "unit" instead of "sign", right? Besides, I'm not sure we want to add a new unit for temperature but I could live with it. Marek, what do you think? See my comments, below. On 09/03/2015 09:43 PM,

Re: [Mesa-dev] [PATCH] fix temperature display sign in hud

Setting max != 100 should remove the "%" sign. Marek On Thu, Sep 3, 2015 at 11:04 PM, Benjamin Bellec wrote: > Yes I meant "unit", "sign" is a bad translation. > In gallium/hud, the temperature is displayed as a percentage, which is quite > confusing. But maybe there is

Re: [Mesa-dev] [PATCH] fix temperature display sign in hud

Yes I meant "unit", "sign" is a bad translation. In gallium/hud, the temperature is displayed as a percentage, which is quite confusing. But maybe there is better and simpler approach to fix that. I do not really understand the code so I tried to copy other part of the code.