Re: [openssl-dev] [openssl.org #4026] patches to eliminate some warnings from clang

2016-01-16 Thread Alessandro Ghedini via RT
Looks like some things are already fixed in master, does this needs any more actions? Cheers ___ openssl-dev mailing list To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-dev

Re: [openssl-dev] [openssl.org #4026] patches to eliminate some warnings from clang

2015-10-05 Thread Kaduk, Ben via RT
Updated patch series to address a couple of comments from Ben (made on now-orphaned github commits). Still available on github at a rebased https://github.com/kaduk/openssl/commits/warning-cleanup . -Ben >From 4b8104dc7fc3450e43edc1a4fd52ece2ed35929a Mon Sep 17 00:00:00 2001 From: Benjamin Kaduk

[openssl-dev] [openssl.org #4026] patches to eliminate some warnings from clang

2015-09-14 Thread Kaduk, Ben via RT
I ran a build using clang's -Weverything to activate all warnings (setting CFLAG in the top-level Makefile and removing all other warning-related flags, especially -Werror). With the system clang on my mac, master at 4d04226c2ec7e7f69f6234def63631648e35e828 with the patch from #3984 gives the

Re: [openssl-dev] [openssl.org #4026] patches to eliminate some warnings from clang

2015-09-14 Thread Kaduk, Ben via RT
Ben's work on df2ee0e27d2db02660c1d15fe6a3e38be9df0a60 made a lot of this stuff redundant; I rebased the branch at https://github.com/kaduk/openssl/commits/warning-cleanup and generated a new warning-cleanup1.patch, attached. Of note: the move of x509v3/ext_dat.h to crypto/include/internal is