Re: Snipping irrelevant text from a discussion (was: "RE: [PATCH 15/16] OMAP3: PM: VP force update method of voltage scaling")

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hey Sergio,

On Fri, Mar 5, 2010 at 8:17 PM, Aguirre, Sergio <saaguirre@xxxxxx> wrote:
> Hey!
>
> You guys take a "chill pill", ok? :) Let's not start shouting at people and point fingers.

I'm not really angry, and I'm not blaming Thara either. I just tried
to do some constructive criticism that was apparently not accepted,
and I decided not to push it any further.

> My 2 cents is that, usually, when discussion starts about a topic, or a
> specific parts of code, you should remove irrelevant text, and substitute
> by a "<snip>" line, just to give reference that some irrelevant text used to
> be there... so if you're interested, just jump to the mailing list archive, and look it out for yourself.

Indeed, specially when dealing with a heated argument, it's sometimes
important to specify where you have snipped text. Although I think
most of the time that's not needed, a "snip" comment is preferable to
dozes or hundreds of extra lines.

> Also, this is completely off topic, and has nothing to do with the original patch subject. Therefore a proper subject rename must happen.

Right.

> If you want to take this topic further, please figure it out between you two
> And, if necessary, the list maintainer, allright?

Yeah, that's something for Tony to decide, but I didn't think it was a
good idea to push for it right now.

Cheers.

-- 
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux