Changeset 96476 in webkit


Ignore:
Timestamp:
Oct 2, 2011 12:41:08 PM (12 years ago)
Author:
jchaffraix@webkit.org
Message:

Remove an unused member in RenderTable::ColumnStruct
https://bugs.webkit.org/show_bug.cgi?id=69119

Reviewed by Darin Adler.

No test as there should be no change in behavior.

The member was added several years ago and its users were removed or changed. The code would
need a refactoring but this would be tricky to get right. This change just removes the member
to reap the short-term benefits.

  • rendering/RenderTable.h:

(WebCore::RenderTable::ColumnStruct::ColumnStruct): Remove anything associated with m_width.

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r96471 r96476  
     12011-10-02  Julien Chaffraix  <jchaffraix@webkit.org>
     2
     3        Remove an unused member in RenderTable::ColumnStruct
     4        https://bugs.webkit.org/show_bug.cgi?id=69119
     5
     6        Reviewed by Darin Adler.
     7
     8        No test as there should be no change in behavior.
     9
     10        The member was added several years ago and its users were removed or changed. The code would
     11        need a refactoring but this would be tricky to get right. This change just removes the member
     12        to reap the short-term benefits.
     13
     14        * rendering/RenderTable.h:
     15        (WebCore::RenderTable::ColumnStruct::ColumnStruct): Remove anything associated with m_width.
     16
    1172011-10-02  Dan Bernstein  <mitz@apple.com>
    218
  • trunk/Source/WebCore/rendering/RenderTable.h

    r95852 r96476  
    127127
    128128    struct ColumnStruct {
    129         enum {
    130             WidthUndefined = 0xffff
    131         };
    132 
    133129        ColumnStruct()
    134130            : span(1)
    135             , width(WidthUndefined)
    136131        {
    137132        }
    138133
    139134        unsigned span;
    140         unsigned width; // the calculated position of the column
    141135    };
    142136
Note: See TracChangeset for help on using the changeset viewer.