Re: [PATCH v2] Remove unused variable in fds/drm.c.
Vinson Lee <[email protected]> Fri, 4 Dec 2015 10:49:42 -0800
| Newsgroups | org.kernel.vger.trinity |
|---|---|
| Message-ID | <CACKvgLHsZMP-XpMEo-FVWiES4d4nw4YBgoybsf7jXEdgUHbUOQ@mail.gmail.com> |
On Fri, Dec 4, 2015 at 10:07 AM, Dave Jones <[email protected]> w= rote: > On Fri, Dec 04, 2015 at 02:18:52AM -0800, Vinson Lee wrote: > > Fix build error with GCC 5.2.1. > > > > CC fds/drm.o > > fds/drm.c: In function =E2=80=98open_drm_fds=E2=80=99: > > fds/drm.c:70:15: error: unused variable =E2=80=98i=E2=80=99 [-Werr= or=3Dunused-variable] > > unsigned int i; > > ^ > > Applied. I wonder why debian's 5.2.1 didn't pick up on this. > Judging by your other patches, I'm guessing this is Fedora's version = ? > > Dave > Yes, the error was from GCC on Fedora. $ gcc -v Using built-in specs. COLLECT_GCC=3Dgcc COLLECT_LTO_WRAPPER=3D/usr/libexec/gcc/x86_64-redhat-linux/5.2.1/lto-wr= apper Target: x86_64-redhat-linux Configured with: ../configure --enable-bootstrap --enable-languages=3Dc,c++,objc,obj-c++,fortran,ada,go,lto --prefix=3D/= usr --mandir=3D/usr/share/man --infodir=3D/usr/share/info --with-bugurl=3Dhttp://bugzilla.redhat.com/bugzilla --enable-shared --enable-threads=3Dposix --enable-checking=3Drelease --enable-multilib --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-linker-hash-style=3Dgnu --enable-plugin --enable-initfini-array --disable-libgcj --with-isl --enable-libmpx --enable-gnu-indirect-function --with-tune=3Dgeneric --with-arch_32=3Di= 686 --build=3Dx86_64-redhat-linux Thread model: posix gcc version 5.2.1 20151104 (Red Hat 5.2.1-5) (GCC) Vinson