From: Stan Shebs <stanshebs@earthlink.net>
To: gdb-patches@sourceware.org
Subject: Re: [PATCH] Doxygenate gdbtypes.h
Date: Tue, 04 Mar 2014 01:18:00 -0000 [thread overview]
Message-ID: <531529D4.3050309@earthlink.net> (raw)
In-Reply-To: <201403040039.s240db0w008027@glazunov.sibelius.xs4all.nl>
On 3/3/14 4:39 PM, Mark Kettenis wrote:
>> Date: Mon, 03 Mar 2014 16:16:29 -0800
>> From: Stan Shebs <stanshebs@earthlink.net>
>>
>> Since minsyms.h and utils.h are churning a bit, I turned my attention to
>> gdbtypes.h. As with defs.h, I've done only minimal content change,
>> although the patch looks large because of many unavoidable paragraph
>> refills.
>>
>> I added an additional block comment at the top that is a bit of general
>> discussion, similar to content that was in gdbint.texinfo. Doxygen
>> starts to win for us here, as not only does it create a separate page
>> for this block, but mentions of symbols become links to definitions and
>> detailed documentation.
>>
>> I'll push this in a few days.
>
> I wish you didn't. This really adds a lot of random garbage all
> through the comments we carefully wrote over the last 20 years.
> It's an act of vandalism.
>
Give the histrionics a rest, please. You claimed it was against policy,
but then you never replied when I asked what specific policy
is being violated. Two extra characters per comment block is not
a "lot of random garbage" or "vandalism" - over-the-top statements
like that are not going to convince anybody of anything.
However, if the majority of maintainers speak up and say that they've
changed their minds and no longer like this idea, I'll certainly
revert what I've done so far and go look for something else to do.
Stan
stan@codesourcery.com
next prev parent reply other threads:[~2014-03-04 1:18 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-04 0:16 Stan Shebs
2014-03-04 0:39 ` Mark Kettenis
2014-03-04 1:18 ` Stan Shebs [this message]
2014-03-04 17:36 ` Ludovic Courtès
2014-03-04 18:18 ` Joel Brobecker
2014-03-04 19:03 ` Stan Shebs
2014-03-04 19:15 ` Eli Zaretskii
2014-03-04 19:51 ` Stan Shebs
2014-03-05 1:17 ` Rob Savoye
2014-03-13 2:38 ` Stan Shebs
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=531529D4.3050309@earthlink.net \
--to=stanshebs@earthlink.net \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox