OSX compile warnings

gcc -O2 -g -Wmissing-prototypes -Wall -pthread   -c client.c
client.c:725:7: warning: variable 'distance' is used uninitialized whenever 'if'
      condition is true [-Wsometimes-uninitialized]
  if (!strcmp(line, "off")) {
      ^~~~~~~~~~~~~~~~~~~~
client.c:737:53: note: uninitialized use occurs here
  msg->data.local.distance = UTI_FloatHostToNetwork(distance);
                                                    ^~~~~~~~
client.c:725:3: note: remove the 'if' if its condition is always false
  if (!strcmp(line, "off")) {
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
client.c:723:18: note: initialize the variable 'distance' to silence this
      warning
  double distance;
                 ^
                  = 0.0
1 warning generated.
> On 9/04/2016, at 2:56 AM, g...@tuxfamily.net wrote:
> 
> This is an automated email from git. It was enerated because a ref
> change was pushed to the repository "chrony/chrony.git".
> 
> The branch, master has been updated
>       via  b252c57a221947d46554a45fc46344a1aeaf210a (commit)
>       via  2aab6a85a456854219d76f8a20ccb790877d4aa2 (commit)
>       via  10719d6d350566acbbad5fddedf40e36f4142347 (commit)
>       via  59938efd23b72d2b689b912bf1f8171233fde8b3 (commit)
>       via  53b15bd5c7f6837441b47874f046f3855a333386 (commit)
>       via  5084a8b34254f9b8f85287c8068cf0d7f23a4246 (commit)
>       via  4d1c79580456e859dbebec91d357da81b1f5719d (commit)
>       via  a9049569af6338d6fd044c8421017280fe0b8743 (commit)
>      from  dec1d2bfb21f9a731b29aceca8b5d9bfe82f83ff (commit)
> 
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
> 
> - Log -----------------------------------------------------------------
> commit b252c57a221947d46554a45fc46344a1aeaf210a
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Fri Apr 8 15:50:27 2016 +0200
> 
>    reference: rework activation of local reference
> 
>    Instead of using a timer for switching the reference to the
>    unsynchronised state (which activates the local reference), check
>    if it should be active when returning the reference parameters.
> 
> commit 2aab6a85a456854219d76f8a20ccb790877d4aa2
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Fri Apr 8 16:10:02 2016 +0200
> 
>    reference: return real sync status in REF_GetReferenceParams()
> 
>    If local reference is active, return normal leap, but unsynchronised
>    status. Update the callers of the function to work with the leap
>    directly and not change their behaviour.
> 
>    REF_IsLocalActive() is no longer needed.
> 
> commit 10719d6d350566acbbad5fddedf40e36f4142347
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Thu Apr 7 17:09:01 2016 +0200
> 
>    reference: report same values in tracking command as in NTP
> 
>    Use REF_GetReferenceParams() in the tracking command to simplify the
>    code and report the same values as what NTP clients of the server see.
> 
>    When the local reference mode is active, this changes the leap status to
>    synchronised and reference time to one second behind current time. When
>    not synchronised, the root delay and root dispersion are now 1 second.
> 
> commit 59938efd23b72d2b689b912bf1f8171233fde8b3
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Fri Apr 8 12:41:13 2016 +0200
> 
>    stubs: add NSR_GetLocalRefid()
> 
> commit 53b15bd5c7f6837441b47874f046f3855a333386
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Thu Apr 7 16:33:23 2016 +0200
> 
>    cmdmon: extend local command to match local directive
> 
> commit 5084a8b34254f9b8f85287c8068cf0d7f23a4246
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Fri Apr 8 12:34:48 2016 +0200
> 
>    reference: clamp local stratum set from cmdmon
> 
> commit 4d1c79580456e859dbebec91d357da81b1f5719d
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Wed Apr 6 16:38:14 2016 +0200
> 
>    cmdparse: check if stratum in local directive is valid
> 
> commit a9049569af6338d6fd044c8421017280fe0b8743
> Author: Miroslav Lichvar <mlich...@redhat.com>
> Date:   Wed Apr 6 15:56:12 2016 +0200
> 
>    cmdmon: remove obsolete definition
> 
> -----------------------------------------------------------------------
> 
> Summary of changes:
> candm.h                        |    9 ++-
> client.c                       |   27 ++++----
> cmdmon.c                       |   12 ++--
> cmdparse.c                     |    3 +-
> doc/chronyc.adoc               |   11 ++--
> ntp_core.c                     |   12 +---
> pktlength.c                    |    3 +-
> refclock.c                     |   12 ++--
> reference.c                    |  134 ++++++++++++----------------------------
> reference.h                    |    3 +-
> reports.h                      |    5 +-
> stubs.c                        |    6 ++
> test/simulation/115-cmdmontime |    2 +-
> 13 files changed, 94 insertions(+), 145 deletions(-)
> 
> 
> hooks/post-receive
> --
> chrony/chrony.git
> 
> -- 
> To unsubscribe email chrony-dev-requ...@chrony.tuxfamily.org with 
> "unsubscribe" in the subject.
> For help email chrony-dev-requ...@chrony.tuxfamily.org with "help" in the 
> subject.
> Trouble?  Email listmas...@chrony.tuxfamily.org.
> 

Bryan Christianson
br...@whatroute.net




--
To unsubscribe email chrony-dev-requ...@chrony.tuxfamily.org with "unsubscribe" 
in the subject.
For help email chrony-dev-requ...@chrony.tuxfamily.org with "help" in the 
subject.
Trouble?  Email listmas...@chrony.tuxfamily.org.

Reply via email to