[LLVMdev] Compile error in include/Support/GraphWriter.h
wendling at isanbard.org
Mon Oct 28 00:51:01 PST 2002
Also sprach Casey Carter:
} Bill? Wendling wrote:
} >Um...was it entirely necessary to issue *8* email messages to the group
} >with mostly single-line fixes instead of just one email with all of the
} >fixes and an explanation for each?
} Actually, yes it was. Proper netiquette when submitting to a technical
} list is to have a single topic per message. This makes it easy to track
} issues individually, without the messiness that occurs from bundling
} several issues together in a single missive. One large email with a big
} lump of diffs is much less clear, and takes substantially more effort to
} parse: Are the patches independent? Which fix corresponds to which
Since it's also proper netiquette not to spam too much, this will be the
end of this thread for me.
What you say is all true. However, the changes you made were very small
(though necessary) and not really subject to the difficulties in parsing
through them that you mentioned. Most consisted of single-line fixes (as
I mentioned above), and a lot of them obvious fixes.
} Being new to this group, I am simply acting as my experience dictates
} and discussing these issues in the way I feel is best. If my
} infamiliarity with the group causes me to occasionally act against what
} is common practice here, I will appreciate it when you inform me that I
} have done so and what the proper approach should be.
I vote for simple/obvious fixes to be combined in one email. I may exist
as a sole entity in feeling this way, but so be it. If others would like
such fixes to be concatenated, feel free to pipe in. If, on the other
hand, people feel that such fixes should be separated, then the consensus
should win out.
Keep in mind, I'm only suggesting this for simple/obvious fixes. Not for
more complex ones which require more detailed descriptions and are more
properly separated into multiple emails, IMHO.
Democracy is fun :-)
|| Bill? Wendling wendling at isanbard.org
More information about the llvm-dev