|In reply to Comment 192 (Don Cox):|
One of the Friedens has already posted to this thread that they shipped the A1 U Boot variant with outstanding warnings and changes to core U Boot code that are only relevant to the A1 hardware. Both examples of terrible software engineering not acceptable for end-user products (especially firmware!) and which, understandably, the U Boot team didn't want to import.
Additionally yes, I have read code written by the Friedens, it was of very poor quality and did not include proper changelogs, explanatory comments etc. It would be enough to fail any introductory course on software engineering, or to be disciplined at a serious (which Hyperion is not) software company.
I've also examined the source to other Amiga software which you yourself, Don, have praised very highly and found it to be even worse. Undocumented numeric constants, poor execution structure, beginners errors with arithmetic and logical operators. Maybe the code hidden away in developer's private archives is of much better quality, but the code I've seen has been almost universally abysmal.