[wp-trac] [WordPress Trac] #54746: Comment Error in wp-login.php

WordPress Trac noreply at wordpress.org
Fri Mar 18 03:17:11 UTC 2022


#54746: Comment Error in wp-login.php
------------------------------------+-------------------------------------
 Reporter:  alkesh7                 |       Owner:  SergeyBiryukov
     Type:  enhancement             |      Status:  assigned
 Priority:  normal                  |   Milestone:  6.0
Component:  Login and Registration  |     Version:
 Severity:  minor                   |  Resolution:
 Keywords:                          |     Focuses:  docs, coding-standards
------------------------------------+-------------------------------------
Changes (by peterwilsoncc):

 * keywords:  has-patch =>
 * owner:  peterwilsoncc => SergeyBiryukov
 * status:  reopened => assigned


Comment:

 Replying to [comment:8 SergeyBiryukov]:
 > I think comments like "Main" have some benefit, as they denote various
 parts of the file, in this case separating function declarations from the
 main part.

 With the additional context they do, on it's own the word main is
 meaningless.

 > Instead of removing them, I think we can convert them to the multi-line
 format per the documentation standards:
 > {{{
 > /*
 >  * Main.
 >  */
 > }}}

 The point is to meet the coding standards by avoiding single line comments
 of that format. Given that the additional context is needed for the
 comment to make sense, I think it best revised as a multi-line comment
 anyway.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/54746#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list