Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.17
    • Fix Version/s: 1.8
    • Component/s: Build-System
    • Security Level: public
    • Labels:
      None
    • Environment:
      macOS 10.12.5
    • CPU Architecture:
      x86-64
    • Kernel:
      OS X (Mach)
    • Userland:
      OS X
    • Distribution:
      Debian
    • Distribution Version:
      OS X 10.10
    • Compiler:
      clang
    • Compiler Version:
      Apple LLVM version 8.1.0 (clang-802.0.42)
    • FreeSWITCH GIT Revision:
      e052ef92187c45cc39a83a6461c818e8288d3cf2
    • GIT Master Revision hash::
      8dece92e5f0c258b75981cfb92a89fb9d0335923

      Description

      Last worked May 11, both todays master and 1.6.17 downloaded today get:

        CC libfreeswitch_la-switch_rtp.lo
      src/switch_rtp.c:1525:8: error: taking the absolute value of unsigned type 'unsigned int' has no effect [-Werror,-Wabsolute-value]
                              if (abs(rtp_session->ts - rtp_session->last_write_ts) > rtp_session->samples_per_interval
                                  ^
      src/switch_rtp.c:1525:8: note: remove the call to 'abs' since unsigned values cannot be negative
                              if (abs(rtp_session->ts - rtp_session->last_write_ts) > rtp_session->samples_per_interval
                                  ^~~
      src/switch_rtp.c:8011:7: error: taking the absolute value of unsigned type 'uint32_t' (aka 'unsigned int') has no effect [-Werror,-Wabsolute-value]
                      if (abs(ts_delta) > rtp_session->samples_per_second * 2) {
                          ^
      src/switch_rtp.c:8011:7: note: remove the call to 'abs' since unsigned values cannot be negative
                      if (abs(ts_delta) > rtp_session->samples_per_second * 2) {
                          ^~~
      2 errors generated.
      make[1]: *** [libfreeswitch_la-switch_rtp.lo] Error 1

        Attachments

          Activity

            People

            • Assignee:
              mikej Mike Jerris
              Reporter:
              mario_fs Mario G
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: