I'd prefer *much* more inline comments as well to make it easier to work on code parts I've never touched before when debugging or adding features as it's very hard to figure out what the methods do and how they interact.
-- Best regards, Alex Am Montag, den 21.06.2010, 16:55 +0200 schrieb J. Shirley: > On Mon, Jun 21, 2010 at 7:43 AM, iain <[email protected]> wrote: > > Ævar Arnfjörð Bjarmason wrote: > >> > >> It depends on what sort of comments you're making. Comments that > >> explain tricky code that help with maintenance down the road are > >> welcome everywhere. If you're just making comments that help someone > >> completely unfamiliar with Catalyst to read the code it'll probably be > >> more distracting than helpful to core devs. > >> > >> > > > > Surely the core devs can work with comments? > > > > Would having more comments not encourage more developers to contribute by > > lowering the barrier for entry? > > > > I don't think I can count as a core developer anymore, but really the > more comments the better (provided they aren't there so you can > practice your buddy career as a novelist). > > If you can't write sane comments, though, I don't think you can write > sane code :) > > _______________________________________________ > List: [email protected] > Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst > Searchable archive: http://www.mail-archive.com/[email protected]/ > Dev site: http://dev.catalyst.perl.org/ *"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"* T-Systems Austria GesmbH Rennweg 97-99, 1030 Wien Handelsgericht Wien, FN 79340b *"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"* Notice: This e-mail contains information that is confidential and may be privileged. If you are not the intended recipient, please notify the sender and then delete this e-mail immediately. *"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"*"* _______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[email protected]/ Dev site: http://dev.catalyst.perl.org/
