syntropy/golang-slides — pull request #1 by surma (2012-09-03 19:37:18+00:00)
Title: Typos 'n stuff

Moin @akrennmair :)

Ich hab' die Präsi mal durchgelesen. Im dem PR hab ich erstmal nur Typos korrigiert – schau dir einfach mal das word-diff an.

Bezüglich Content hab' ich nur ein paar wenige Anmerkungen:

* Ich würde *nicht* auf `[]byte` Basis arbeiten. Führe ruhig direkt am Anfang des Chatprogramms `bufio` ein und/oder nutze `io.WriteString(io.Writer, string)`. Das verbirgt die Slice/String “Problematik” und halte ich für didaktisch sinnvoll.
* Stell an einer angebrachten Stelle das Fan-In/-Out Pattern vor. Man brauch es andauernd – auch hier – und kann so mehrere Zeilen auf einmal mental abhaken.
* Ich hab dir auf dem [Chat-Server](https://github.com/akrennmair/telnet-chat) auch einen PR aufgemacht, wo den Code mal ein wenig refactored habe. Mir geht's hauptsächlich um die `handleConnection()`, die mir einfach zu lang war ;) Ist natürlich alles wieder zur diskussion gestellt.

Der @dtl wird seinen Senf auch noch irgendwie abgeben ^^