Rockbox mail archiveSubject: Re: Random numbers et al
From: Björn Stenberg (bjorn_at_haxx.se)
Blue Chip wrote:
You appear to have much to offer. But rewriting existing code in assembler is not a top priority for the project. I'm sure you have read the docs/CONTRIBUTING file:
If you rewrote, say, sprintf in assembler we'd have a much harder time adding new formatting options to it, for example. Hence the doubt: "Is it a benefit?"
> There are zero docs on how the firmware works or how it "hangs together"
No comments? We must be reading different sources. In fact, we have received much praise about how our code is simple and easy to follow.
However, I admit we are severely lacking in the documentation department. This is a common problem in voluntary projects, since it's often more rewarding to write new code than to write documentation about the old. That, plus documentation quickly becomes outdated. This is one area where we need help.
> I thought that I would offer you the benefit of my skills
Your skills are most welcome. But you must expect people to question your assumptions, especially when you come out with all guns blazing ("a horribly complex and memory hungry block of code").
This is a project with many highly skilled people, and hence many great egos. We should all strive to act and speak in a humble fashion, and the general discussion will be more pleasant.
> You can imagine how I felt when it was suggested that my skills would
I just checked the log. It looks like a simple misunderstanding. Nobody talked about flashing a cursor. adi|work mentioned a problem which might actually spark your interest:
There is a general desire to change the current cursor into an inverted line, but the only attempt so far suffered from poor performance while scrolling through a file list, and thus was rejected. It's a tricky problem due to the lcd memory layout, our dynamic font system and the fact that lcd update is done over a rather slow serial link (written in assembler, by the way).
In closing, this is a voluntary project. Everyone does whatever tickles their fancy. Therefore, we don't assign tasks to people. We have a feature request tracker (http://rockbox.haxx.se/requests.shtml) and I keep a loosely written TODO (http://rockbox.haxx.se/TODO), but nobody decides who does what.
-- Björn Rockbox project leader
Page was last modified "Jan 10 2012" The Rockbox Crew