[FS-7965] Warning made error by gcc in conference_video.c Created: 14/Aug/15  Updated: 17/Aug/15  Resolved: 17/Aug/15

Status: Resolved
Project: FreeSWITCH
Component/s: mod_conference
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Minor
Reporter: Trever L. Adams Assignee: Mike Jerris
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment: Fedora 22 with all updates

Attachments: Text File fix.patch    
CPU Architecture:
uname: Linux HighCountry.SnowyRiver.SapphireSunday.org 4.1.4-200.fc22.x86_64 #1 SMP Tue Aug 4 03:22:33 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Distribution Version:
Debian 8 jessie
lsb_release: SB Version: :core-4.1-amd64:core-4.1-noarch:cxx-4.1-amd64:cxx-4.1-noarch:desktop-4.1-amd64:desktop-4.1-noarch:languages-4.1-amd64:languages-4.1-noarch
Distributor ID: Fedora
Description: Fedora release 22 (Twenty Two)
Release: 22
Codename: TwentyTwo
Compiler Version: gcc version 5.1.1 20150618 (Red Hat 5.1.1-4) (GCC)
FreeSWITCH GIT Revision: dff03e35cfd532da630f2d5b802fce36c07d4feb
GIT Master Revision hash:: dff03e35cfd532da630f2d5b802fce36c07d4feb

gcc with the settings between FreeSWITCH make files and system defaults on Fedora 22 complains about the lack of parentheses in the if around line 2102.

Comment by Trever L. Adams [ 14/Aug/15 ]
A patch which seems to solve the warning/error and keep the code functioning as intended.
Comment by Mike Jerris [ 14/Aug/15 ]
patch looks right, can you create a pull request for this please.
Comment by Trever L. Adams [ 14/Aug/15 ]
I am afraid I don't know how to do this yet.
Comment by Brian West [ 17/Aug/15 ]
Comment by Trever L. Adams [ 17/Aug/15 ]
Thank you!
Generated at Tue May 21 03:40:47 CDT 2019 using Jira 8.1.0#801000-sha1:2e1cd1bb771978cda2c5e8f3f10539ab180613f6.