[wp-trac] [WordPress Trac] #40293: Comment template - warning

WordPress Trac noreply at wordpress.org
Wed Mar 29 15:05:19 UTC 2017


#40293: Comment template - warning
-------------------------------------------------+-------------------------
 Reporter:  victorfreitas                        |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Comments                             |  Review
 Severity:  normal                               |     Version:  trunk
 Keywords:  has-patch needs-unit-tests           |  Resolution:
  reporter-feedback                              |     Focuses:
-------------------------------------------------+-------------------------

Comment (by victorfreitas):

 Thanks @thamaraiselvam you're right.
 By default comment args offset is empty even.


 Replying to [comment:1 thamaraiselvam]:
 > Yes, It could lead to divide-by-zero condition in some scenario and
 @victorfreitas I have removed the variable
 > {{{
 > $offset
 > }}}
 >
 > to reduce unnecessary memory usages because it's just one-time usage.

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


More information about the wp-trac mailing list