Quoting Henning Schild (2019-01-29 10:10:50)
...
> > >> Do you think that refactoring should be part of this patchset?  
> > > 
> > > I am not sure, just wanted to write it down. On the one hand it
> > > would be nice to solve all drive-by issues as we go. On the other
> > > hand that will slow down development and impose on contributors.
> > > So feel free to ignore my comments, the inconsistencies are not your
> > > fault and asking you to fix them would be too much.  
> > 
> > Maybe we should open issues for all those inconsistencies and 
> > annoyances, so that they can be fixed or at least be documented.
> 
> We do not use issues, so we would need another way to track not fully
> resolved discussions from the list. But i agree, it would be nice to
> keep track of things in a structured way to fix them eventually.
> Maybe a TODO.md?

I am not really a fan of TODO lists that are checked into the VCS.

I would find it very strange to send patches to the TODO list... I don't
even know what the process there would be? Should I first send a patch
to the TODO list and then have discussions about it and afterwarts start
working on patches in isar?

I rather have some uncoupled list of TODO items somewhere without the
patch/review overhead.

> > At some point making a complete code review (not just patches) on
> > isar would be great, just to find and fix those.
> 
> I thought the same, maybe an Isar hackathon.

Maybe... I am not a hackathon expert. I don't know if quality codes
comes out of that...

Claudius

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de

           PGP key: 6FF2 E59F 00C6 BC28 31D8 64C1 1173 CB19 9808 B153
                             Keyserver: hkp://pool.sks-keyservers.net