<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">With diff you at least would more easily find what the author changed.<div class="">With patch I could easily re-implement my changes to several modules</div><div class="">(Personally I have to add everytime the cursor steering in TextFrames.Mod and my Swiss German keyboard in Input.Mod...)</div><div class=""><br class=""></div><div class="">Multiplier:<div class=""><div class="">One version does not use the multiplier capabilities of the FPGA, pure shifting and adding.</div><div class="">The other version uses the FPGA multiplier blocks.</div><div class="">Described in 16.2.2: <a href="https://www.inf.ethz.ch/personal/wirth/ProjectOberon/PO.Computer.pdf" class="">https://www.inf.ethz.ch/personal/wirth/ProjectOberon/PO.Computer.pdf</a></div><div class=""><br class=""></div><div class="">Mouse:</div><div class="">two drivers for two different Mouse HW: MouseP based on PS/2 interface, MouseX for historic Lilith mouse</div><div class="">described in 17.2.1+2: <a href="https://www.inf.ethz.ch/personal/wirth/ProjectOberon/PO.Computer.pdf" class="">https://www.inf.ethz.ch/personal/wirth/ProjectOberon/PO.Computer.pdf</a></div><div class=""><br class=""></div><div class="">br</div><div class="">Jörg<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">Am 25.10.2018 um 08:05 schrieb Skulski, Wojciech <<a href="mailto:skulski@pas.rochester.edu" class="">skulski@pas.rochester.edu</a>>:</div><br class="Apple-interchange-newline"><div class=""><div class="">Jörg<br class=""><br class=""><blockquote type="cite" class="">Indeed, version control is not one of NWs strengths :-)<br class="">I lately had the idea to implement „diff" and „patch“ in Oberon.<br class=""></blockquote><br class="">Version control will not provide insight into the intent of the writer. For example, I am curious what is the difference between RISC5.v dated 31.8.2018 and RISC5a.v dated 1.9.2018. Both can be fetched from the same page <a href="http://www.inf.ethz.ch/personal/wirth/ProjectOberon/index.html" class="">www.inf.ethz.ch/personal/wirth/ProjectOberon/index.html</a>.<br class=""><br class=""> Fortunately, there are some comments in that code:<br class=""><br class=""> RISC5.v dated 31.8.2018 //with interrupt and floating-point<br class=""> RISC5a.v dated 1.9.2018 //no interrupt, no floating-point<br class=""><br class="">The former is accompanied by Multiplier.v dated 14.9.2015. <br class="">The latter is accompanied by Multiplier1.v dated 29.4.2011<br class=""><br class="">So here I have a question: why two multipliers, and why their codes are different? Would be good to see a comment on the website. But there is no comment. Just a horizontal line.<br class=""><br class="">Even more puzzling is that RISC5.v is accompanied by MouseP.v, while RISC5a.v is accompanied by MouseX.v, which are again quite different. Another puzzle.<br class=""><br class="">The term "accompanied by" is my own interpretation of the horizontal lines dividing this web page. It would be good to know why the Verilog modules have been grouped this particular way. Is it true that MouseP.v needs floating point, while MouseX.v does not? Or is it an accident that they have been parted by a horizontal line?<br class=""><br class="">So the patch and the diff are not the only tools which we would need. Comments would be even more useful.<br class=""><br class="">How can I force my students to comment their source code when they can see this situation?<br class=""><br class="">W<br class=""><br class=""><br class=""><br class=""><br class=""><br class=""><br class=""><br class="">--<br class=""><a href="mailto:Oberon@lists.inf.ethz.ch" class="">Oberon@lists.inf.ethz.ch</a> mailing list for ETH Oberon and related systems<br class=""><a href="https://lists.inf.ethz.ch/mailman/listinfo/oberon" class="">https://lists.inf.ethz.ch/mailman/listinfo/oberon</a><br class=""></div></div></blockquote></div><br class=""></div></div></div></body></html>