Comment 4 for bug 1501877

Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote : Re: [Bug 1501877] Re: Clang static analyzer report

On Thu, Oct 08, 2015 at 06:37:46PM -0000, Sergey Stepanov wrote:
> Hello
>
> > I've merged your branch into master
>
> It was too premature. I planned to put fixups in that branch, so you or
> other maintainer can review them and cherry-pick particular commits that
> worth it. You see, I, as a new developer, have to spend a plenty of time
> to dive into sources, so I concidered that fighting these simple bugs
> would be a good start. Anyway, done is done, thank you.
I don't think so. I reviewed them and merged one by one. Or do you think
I had to wait until you fix all the several hundreds of the declared
bugs? :-P I haven't removed the branch, so please continue your work.
>
> > Does the fixed block in commit 4cc1c7c do anything?
>
> I thought this commit will fix one annoying bug I see every day, but I
> failed.
>
> > Do you consider it worth to have it opened here, which might continue
> forever while we are writing code?
>
> Well, why not? There is the receipe to collect bugs, the thorough list
> of automatically found bugs. Maybe they are deals with already
> registered bugs. (Or do you mean, that this bugreport should be opened
> in other LP project?)
No, I don't :) However, this is just a routine work that devs do if they
want to. AFAIUI, bugtrackers are more about user annoying bugs and
wishlists. And one of their goals is to show users the progress we have.
>
> > Wouldn't it be better to just fix the bugs you see in the master
> branch?
>
> Maybe yes, maybe no, I don't know. You merged the obvious and trivial
> pathes, but others are not so trivial, and may require more work (or
> rework). So, this branch was a sandbox for my patches.

OK, thanks,
  Vladimir