Merge glib branch to master

Allen Winter winter at kde.org
Wed Jun 29 14:11:01 PDT 2016


On Wednesday, June 29, 2016 02:37:13 PM Milan Crha wrote:
> On Sun, 2016-06-26 at 11:28 -0400, Allen Winter wrote:
> > Once this lands and we think things still work ok, I will be
> > adjusting the CMake buildsystem
> > and start requiring CMake version 3.0.2 or higher.
> 
> 	Hi,
> that looks pretty good. Thanks for the inclusion of the libical-glib
> in the upstream version.
> 
> I re-run static analysers on the git master at commit b2293fa7f and
> uploaded the results here:
> https://people.gnome.org/~mcrha/libical/libical-master-commit-b2293fa7f.html
> 
what tool is generating the warnings for C++, for example:

Error: RW.NO_CONSTRUCTOR_FOR_CONVERSION: [#def22]
src/libical/icalparameter_cxx.cpp:105: no_constructor_for_conversion: no suitable constructor exists to convert from "char *" to "std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>"

is that cppcheck?  or something else?
I'd like to be able to run the tool locally 


> I turned on some more options, thus it's more picky, but not everything
> might be considered an issue, some are pure hints.
> 	Thanks again and bye,
> 	Milan
> 
> _______________________________________________
> libical-devel mailing list
> libical-devel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/libical-devel




More information about the libical-devel mailing list