krazy2 stuff done for libqapt changes, what else?
themanchicken at gmail.com
Mon Dec 16 12:20:34 UTC 2013
Yeah, it's a big change. I reworked how sources files are tracked, I now
allow arbitrary sources file loading, and I've added a bunch of testing.
Also I fixed a bunch of warnings.
I'm still getting deprecation warnings from libapt-pkg, but I don't really
know what to do with those, and those were already there when I started. I
enabled Wall my own environment and got everything but those.
I'll look at clang, too.
On Dec 16, 2013 5:31 AM, "Harald Sitter" <apachelogger at ubuntu.com> wrote:
> On Mon, Dec 16, 2013 at 5:14 AM, Michael Stemle <themanchicken at gmail.com>
> > Hey all, when I submitted some of my previous changes I remember there
> > a handful of automated tools used to find potential issues with my code.
> > of those was Krazy2. I've now gone through most of my code with krazy2,
> > now I'm wondering if there are other tools I need to run my code through.
> krazy checks pretty much everything one should care about from a KDE
> POV (well, the stuff that is checkable anyway). running the clang
> static analyzer usually is a thing I tend to do .
> > I believe that the correct path for submitting these changes is
> > them to the KDE Review Board, please let me know if I'm wrong. I'm
> trying to
> > clean things up as much as possible before sending it, so far that has
> > looked like me going over the code again and again with various tools,
> > reviewing my test cases, etc.
> that depends how big the overall change is. I'd ask apol what he would
> like. I for one do prefer to review an actual git branch if a diff
> exceeds 1000 lines and reviewboard for everything else.
> > If someone has time to review my cmake changes that would be helpful. I'm
> > pretty weak with cmake, and while my build seems to be working, most of
> > cmake changes felt like a bit too much trial-and-error.
> > My repository is here if you're interested in looking:
> > https://github.com/manchicken/libqapt.git
> on my todo somehwere :S
>  http://clang-analyzer.llvm.org/
> kubuntu-devel mailing list
> kubuntu-devel at lists.ubuntu.com
> Modify settings or unsubscribe at:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kubuntu-devel