Closed Bug 266113 Opened 20 years ago Closed 20 years ago

Merge of nsXFormsMDG and nsXFormsMDGEngine

Categories

(Core Graveyard :: XForms, defect)

Product:
Core Graveyard
Component:
XForms
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: allan, Assigned: allan)

References

(
URL
)

Details

Attachments

(1 file, 2 obsolete files)

Removed IDL and updated to current CVS
55.09 KB, patch
bryner
: review+
Details | Diff | Splinter Review 
As we have talked about before, nsXFormsMDG and nsXFormsEngine should be merge
into one interface. They were only split for historical reasons.
Attached patch Patch for the above (obsolete) — — Splinter Review 
This patch merges the two classes, and also implements an IDL interface
(nsIXFormsMDG) for the MDG engine.
Status: NEW → ASSIGNED
Attachment #163416 - Flags: review?(bryner)
Attached patch Updated to current CVS (obsolete) — — Splinter Review 

        Attachment #163416 -
        Attachment is obsolete: true

    
  

        Attachment #163416 -
        Flags: review?(bryner)

    
  

        Attachment #164167 -
        Flags: review?(bryner)

    
    

    
  
Comment on attachment 164167 [details] [diff] [review]
Updated to current CVS

Let's not add an IDL interface if we don't need it for some reason.  It adds
vtable bloat, and also forces methods to use out-params instead of returning
values.

        Attachment #164167 -
        Flags: review?(bryner) → review-

    
    

    
  


  

        Attachment #164167 -
        Attachment is obsolete: true

    
  

        Attachment #165131 -
        Flags: review?(bryner)

    
  
Blocks: 265467

        Attachment #165131 -
        Flags: review?(bryner) → review+

    
    

    
  
checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: