Closed Bug 226757 Opened 21 years ago Closed 21 years ago

fixed table layout strategy strange array handling

Categories

(Core :: Layout: Tables, defect)

Product:
Core
Component:
Layout: Tables
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bernd_mozilla, Assigned: bernd_mozilla)

Details

Attachments

(1 file)

patch
918 bytes, patch
dbaron
: review+
dbaron
: superreview+
Details | Diff | Splinter Review 
I dont understand the purpose of 
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/html/table/src/FixedTableLayoutStrategy.cpp&mark=98,100#95
The only reason that I see is to win the obfuscated C contest.
Attached patch patch — — Splinter Review 

    
  
Assignee: table → bernd_mozilla

    
    

    
  
Comment on attachment 136303 [details] [diff] [review]
patch

Boris could you explain me why one would do this.

        Attachment #136303 -
        Flags: superreview?(bz-vacation)

        Attachment #136303 -
        Flags: review?(bz-vacation)

        Attachment #136303 -
        Flags: superreview?(bz-vacation)

        Attachment #136303 -
        Flags: superreview+

        Attachment #136303 -
        Flags: review?(bz-vacation)

        Attachment #136303 -
        Flags: review+

    
    

    
  
Nope.  I have no idea why one would do that.  It's totally bogus...

    
    

    
  
fix checked in
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: