Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 01 Jan 2018 13:06:58 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 224816] emulators/open-vm-tools: Fix build with clang 6.0.0
Message-ID:  <bug-224816-13@https.bugs.freebsd.org/bugzilla/>

next in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D224816

            Bug ID: 224816
           Summary: emulators/open-vm-tools: Fix build with clang 6.0.0
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: Individual Port(s)
          Assignee: jpaetzel@FreeBSD.org
          Reporter: dim@FreeBSD.org
          Assignee: jpaetzel@FreeBSD.org
             Flags: maintainer-feedback?(jpaetzel@FreeBSD.org)

Created attachment 189312
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D189312&action=
=3Dedit
Fix tautological (endless) loop in vixTools.c

While testing the new clang600-import branch (see bug 224669), I noticed th=
at
open-vm-tools fails to compile, due to a new warning in clang 6.0.0:

--- libvix_la-vixTools.lo ---
vixTools.c:7333:22: error: result of comparison 'int' <=3D 4294967295 is al=
ways
true [-Werror,-Wtautological-constant-compare]
   for (var =3D 0; var <=3D 0xFFFFFFFF; var++) {
                 ~~~ ^  ~~~~~~~~~~
1 error generated.

Here is a patch to fix that particular error, and this also avoids an endle=
ss
loop in practice.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-224816-13>