On 2016-02-15 17:29:15 [+0100], Louis Bouchard wrote: > In 0007-fix-ssize_t-size_t-off_t-printf-modifier.patch, the patch on > sigtool/sigtool.c had the following cast : (unsigned long)strlen > > > + mprintf("Signature received (length = %" _sizet ")\n", > > > > + (unsigned long)strlen(buff) - 10); > > > > That cast has disapeared from the renamed patch > fix-ssize_t-size_t-off_t-printf-modifier.patch : > > > + mprintf("Signature received (length = %" _sizet ")\n", > > > > + strlen(buff) - 10); > > > > Was this removal intentional ? If not, I would be happy to file a bug for this > including a refresh of the patch since I intend to add it an an Ubuntu delta. > > If better otherwise, I'll drop that delta on my side.
strlen() returns size_t and the size-modifier _sizet is for size_t so there is no need for a type cast here. There should also be no warning here at compile time. > TIA, > > Kind regards, > > ...Louis Sebastian _______________________________________________ Pkg-clamav-devel mailing list Pkg-clamav-devel@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-clamav-devel