Skip to content

Comments with lots of /s should not have spaces inserted #652

Closed

Description

@nrc

e.g.,

-////////////////////////////////////////////////////////////////////////////////
-// Basic slice extension methods
-////////////////////////////////////////////////////////////////////////////////
+/// /////////////////////////////////////////////////////////////////////////////
+/// Basic slice extension methods
+/// /////////////////////////////////////////////////////////////////////////////

Note that we're changing a regular comment to a doc comment here too

Activity

  1. added
    C-bugCategory: this is a bug; use also I-* labels for specific bug kinds, e.g. I-non-idempotency or I-ICE
    on Nov 23, 2015
  2. added this to the milestone on Apr 6, 2016
  3. self-assigned this
    on Apr 18, 2016
  4. marcusklaas commented on Apr 18, 2016

    @marcusklaas
    Contributor

    My lord, I forgot how much of a mess the comment code is 😱

  5. crumblingstatue commented on May 13, 2016

    @crumblingstatue
    Contributor

    Might not be the same issue, but we encountered this in rust-sfml.

    The normal commented out code gets changed to doc comments.

     /// Shapes implementations using reference counting to manage shared resources
    -// pub mod rc {
    -//     pub use graphics::circle_shape::rc::CircleShape;
    -//     pub use graphics::rectangle_shape::rc::RectangleShape;
    -//     pub use graphics::convex_shape::rc::{ConvexShape, ConvexShapePoints};
    -//     pub use graphics::shape::rc::Shape;
    -//     pub use graphics::shader::rc::Shader;
    -//     pub use graphics::text::rc::Text;
    -//     pub use graphics::sprite::rc::Sprite;
    -//     pub use graphics::render_states::rc::RenderStates;
    -// }
    +/// pub mod rc {
    +///     pub use graphics::circle_shape::rc::CircleShape;
    +///     pub use graphics::rectangle_shape::rc::RectangleShape;
    +///     pub use graphics::convex_shape::rc::{ConvexShape, ConvexShapePoints};
    +///     pub use graphics::shape::rc::Shape;
    +///     pub use graphics::shader::rc::Shader;
    +///     pub use graphics::text::rc::Text;
    +///     pub use graphics::sprite::rc::Sprite;
    +///     pub use graphics::render_states::rc::RenderStates;
    +/// }

    EDIT: This issue does not seem to appear using the rustfmt master branch.

  6. srinivasreddy commented on Jul 17, 2016

    @srinivasreddy
    Contributor

    As said in #1094 I am taking this. Thanks @nrc

  7. added
    only-with-optionRequires a non-default option value to reproduce
    and removed
    only-with-optionRequires a non-default option value to reproduce
    on Jan 16, 2017
  8. topecongiro commented on May 28, 2017

    @topecongiro
    Contributor

    This issue still get reproduced when normalize_comments = true.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-commentsArea: commentsC-bugCategory: this is a bug; use also I-* labels for specific bug kinds, e.g. I-non-idempotency or I-ICEonly-with-optionRequires a non-default option value to reproduce

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions