On 2016-06-08 11:15, Sönke Ludwig wrote:

I generally really do appreciate your critique, but without backing
reasons it doesn't really have a constructive effect.

Two good properties about restricting to /+ +/ is that it's still
possible to put something else in front of it, and that it stands out
from the usual /* */ comments. Sure arguments can be made for supporting
all comment types, but it shouldn't be forgotten that in the end this is
a question of absolutely minimal impact.

Just to be clear: If anyone has a good argument for supporting more/all
comment types and there are no equally good arguments against it, please
go ahead and implement it and it will be included. Let's just make this
a quick decision, because changing it later on will mean breakage no
matter how it's done.

It's just that since the language support other styles of comments one could think that all comments are supported and it will cause confusion if only one style is supported. Otherwise it might be better to use a UDA, if possible.

If one reads the documentation and copy pastes and example the user will most likely get it right. But if you have a conversation saying something like "it's possible to put the content of dub.json inline in the source code, just put it in a comment" then someone will for sure try using a comment style that is not supported.

The documentation needs to be very clear that only one type comments are supported, if that's the conclusion.

--
/Jacob Carlborg
  • Beta release DUB 1.0.0-bet... Sönke Ludwig via Digitalmars-d-announce
    • Re: Beta release DUB ... Edwin van Leeuwen via Digitalmars-d-announce
    • Re: Beta release DUB ... wobbles via Digitalmars-d-announce
    • Re: Beta release DUB ... Luis via Digitalmars-d-announce
    • Re: Beta release DUB ... Marc Schütz via Digitalmars-d-announce
      • Re: Beta release ... Sönke Ludwig via Digitalmars-d-announce
        • Re: Beta rele... Jacob Carlborg via Digitalmars-d-announce
          • Re: Beta ... Sönke Ludwig via Digitalmars-d-announce
            • Re: ... Jacob Carlborg via Digitalmars-d-announce
              • ... Sönke Ludwig via Digitalmars-d-announce
                • ... Jacob Carlborg via Digitalmars-d-announce
                • ... burjui via Digitalmars-d-announce
                • ... Sönke Ludwig via Digitalmars-d-announce
                • ... Mike Parker via Digitalmars-d-announce
                • ... Steven Schveighoffer via Digitalmars-d-announce
                • ... Jacob Carlborg via Digitalmars-d-announce
                • ... Sönke Ludwig via Digitalmars-d-announce
                • ... Rory McGuire via Digitalmars-d-announce
                • ... Sönke Ludwig via Digitalmars-d-announce
                • ... Steven Schveighoffer via Digitalmars-d-announce
                • ... Sönke Ludwig via Digitalmars-d-announce

Reply via email to