Limin Wang (12020-03-29):
> If av_sscanf failed,  s->weights[i] is invalid data and can't be used
> further.

I think you are mistaken: if av_sscanf() fails, s->weights[i] stays
unchanged: it was 0, it's still 0.

> As the old code allow to use the last weights, so I use 
> continue to avoid invalid access only.

It's not invalid, it's 0.

> Do you think it's better to
> print out one warning message?

With this correction, I think this change is not necessary at all.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to