Opened 12 months ago

Closed 5 weeks ago

#6749 closed defect (invalid)

ffmpeg-3.4/libavcodec/eatqi.c:115: suspicious assignment ?

Reported by: dcb Owned by:
Priority: normal Component: undetermined
Version: unspecified Keywords:
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

ffmpeg-3.4/libavcodec/eatqi.c:115]: (style) int result is assigned to long variable.

Source code is

const int64_t qscale = (215 - 2*quant)*5;

maybe better code

const int64_t qscale = (215L - 2*quant)*5;

Change History (4)

comment:1 Changed 12 months ago by jkqxz

What difference would it make? quant is not large.

comment:2 Changed 12 months ago by dcb

So would it be a good idea to make qscale into an int ?

comment:3 Changed 12 months ago by heleppkes

Its fine the way it is.

comment:4 Changed 5 weeks ago by cehoyos

  • Component changed from avcodec to undetermined
  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.