Changeset 141 in dev

Timestamp:

Jun 15, 2007, 1:12:14 PM (17 years ago)

Author:

dai_9181

Message:

CClass::vtblNumをリファクタリングした。
インターフェイスを継承したとき、Objectクラスがインターフェイスの後ろに配置されてしまうバグを修正（どんなときも、Objectクラスが一番最初に継承されるべき）

Files:

• BasicCompiler32/Compile_ProcOp.cpp (modified) (1 diff)
• BasicCompiler64/Compile_ProcOp.cpp (modified) (1 diff)
• BasicCompiler_Common/Class.cpp (modified) (13 diffs)
• BasicCompiler_Common/Class.h (modified) (3 diffs)
• BasicCompiler_Common/DebugMiddleFile.cpp (modified) (2 diffs)
• BasicCompiler_Common/common.h (modified) (1 diff)

BasicCompiler32/Compile_ProcOp.cpp

@@ -515 +515 @@
 
             //仮想関数テーブルを初期化
-            if(pobj_CompilingClass->vtbl_num&&
-                pobj_CompilingClass->IsAbstract()==false){
+            if( pobj_CompilingClass->IsExistVirtualFunctions()
+                && !pobj_CompilingClass->IsAbstract() ){
                     //関数テーブルに値をセット
                     int offset = (int)pobj_CompilingClass->GetVtblGlobalOffset();

BasicCompiler64/Compile_ProcOp.cpp

@@ -567 +567 @@
 
             //仮想関数テーブルを初期化
-            if(pobj_CompilingClass->vtbl_num&&
-                pobj_CompilingClass->IsAbstract()==false){
+            if( pobj_CompilingClass->IsExistVirtualFunctions()
+                && !pobj_CompilingClass->IsAbstract() ){
                     //関数テーブルに値をセット
                     int offset = (int)pobj_CompilingClass->GetVtblGlobalOffset();

BasicCompiler_Common/Class.cpp

@@ -22 +22 @@
     , classType( Class )
     , pobj_InheritsClass( NULL )
-    , vtbl_num( 0 )
+    , vtblNum( 0 )
     , iAlign( 0 )
     , vtbl_offset( -1 )
@@ -90 +90 @@
         const CClass *pInheritsClass = pobj_DBClass->Find(temporary);
         if( !pInheritsClass ){
-            SetError(106,temporary,i);
+            SetError(106,temporary,nowLine);
             return false;
         }
 
         if( pInheritsClass->IsInterface() ){
-            // インターフェイスを継承する
-            if( !InheritsInterface( *pInheritsClass, nowLine ) ){
-                return false;
-            }
+            // インターフェイスはあとで継承する
         }
         else if( pInheritsClass->IsClass() ){
@@ -127 +124 @@
         }
 
-        if( !InheritsClass( *pObjectClass, i ) ){
+        if( !InheritsClass( *pObjectClass, nowLine ) ){
             return false;
         }
+    }
+
+    i=0;
+    while( true ){
+
+        char temporary[VN_SIZE];
+        for( int i2=0;; i++, i2++ ){
+            if( inheritNames[i] == '\0' || inheritNames[i] == ',' ){
+                temporary[i2] = 0;
+                break;
+            }
+            temporary[i2] = inheritNames[i];
+        }
+
+        //継承元クラスを取得
+        const CClass *pInheritsClass = pobj_DBClass->Find(temporary);
+        if( !pInheritsClass ){
+            SetError(106,temporary,nowLine);
+            return false;
+        }
+
+        if( pInheritsClass->IsInterface() ){
+            // インターフェイスを継承する
+            if( !InheritsInterface( *pInheritsClass, nowLine ) ){
+                return false;
+            }
+        }
+        else if( pInheritsClass->IsClass() ){
+            // クラスはさっき継承した
+        }
+        else{
+            SetError(135,NULL,nowLine);
+            return false;
+        }
+
+        if( inheritNames[i] == '\0' ){
+            break;
+        }
+        i++;
     }
 
@@ -189 +225 @@
 
     //仮想関数の数
-    vtbl_num += inheritsClass.vtbl_num;
+    AddVtblNum( inheritsClass.GetVtblNum() );
 
     //継承先のクラスをメンバとして保持する
@@ -235 +271 @@
     }
 
-    interfaces.push_back( InheritedInterface( const_cast<CClass *>(&inheritsInterface), vtbl_num ) );
+    interfaces.push_back( InheritedInterface( const_cast<CClass *>(&inheritsInterface), vtblNum ) );
 
     //仮想関数の数
-    vtbl_num += inheritsInterface.vtbl_num;
+    AddVtblNum( inheritsInterface.GetVtblNum() );
 
     return true;
@@ -308 +344 @@
 int CClass::GetMemberOffset( const char *memberName, int *pMemberNum ) const
 {
-    int i2,offset;
+    int i2;
 
     //仮想関数が存在する場合は関数リストへのポインタのサイズを追加
-    if(vtbl_num) offset=PTR_SIZE;
-    else offset=0;
+    int offset = IsExistVirtualFunctions() ? PTR_SIZE : 0;
 
     int alignment;
@@ -381 +416 @@
 int CClass::GetAlignment() const
 {
-    int alignment,member_size;
-
-    if(vtbl_num) alignment=PTR_SIZE;
-    else alignment=0;
+    //仮想関数が存在する場合は関数リストへのポインタのサイズを追加
+    int alignment = IsExistVirtualFunctions() ? PTR_SIZE : 0;
 
     BOOST_FOREACH( CMember *pMember, dynamicMembers ){
+        int member_size;
         if(pMember->GetType().IsStruct()){
             //メンバクラスのアラインメントを取得
@@ -431 +465 @@
 
     UserProc **ppsi;
-    ppsi=(UserProc **)HeapAlloc(hHeap,0,vtbl_num*sizeof(GlobalProc *));
+    ppsi=(UserProc **)HeapAlloc(hHeap,0,GetVtblNum()*sizeof(GlobalProc *));
 
     //関数テーブルに値をセット
@@ -452 +486 @@
     }
 
-    vtbl_offset=dataTable.AddBinary((void *)ppsi,vtbl_num*sizeof(LONG_PTR));
-
-    for( int i=0; i < vtbl_num; i++ ){
+    vtbl_offset=dataTable.AddBinary((void *)ppsi,GetVtblNum()*sizeof(LONG_PTR));
+
+    for( int i=0; i < GetVtblNum(); i++ ){
         pobj_Reloc->AddSchedule_DataSection(vtbl_offset+i*sizeof(LONG_PTR));
     }
@@ -469 +503 @@
 
     int i;
-    for(i=0;i<vtbl_num;i++){
+    for(i=0;i<GetVtblNum();i++){
         GlobalProc *pUserProc;
         pUserProc=(GlobalProc *)pVtbl[i];
@@ -938 +972 @@
 
     if( isVirtual ){
-        pobj_c->vtbl_num++;
+        pobj_c->AddVtblNum( 1 );
     }
 
@@ -1081 +1115 @@
 
                 //仮想関数の数を初期化
-                pobj_c->vtbl_num=0;
+                pobj_c->SetVtblNum( 0 );
             }
 Interface_InheritsError:
@@ -1214 +1248 @@
 
                 // 仮想関数の数を初期化
-                pobj_c->vtbl_num = 0;
+                pobj_c->SetVtblNum( 0 );
             }
             else{

BasicCompiler_Common/Class.h

@@ -65 +65 @@
     int ConstructorMemberSubIndex;
     int DestructorMemberSubIndex;
+    int vtblNum;                    // 仮想関数の数
 
     // 静的メソッド
@@ -84 +85 @@
     const CClass *pobj_InheritsClass;
 
-    //仮想関数の数
-    int vtbl_num;
-
     //アラインメント値
     int iAlign;
@@ -156 +154 @@
     //デストラクタ メソッドを取得
     const CMethod *GetDestructorMethod() const;
+
+    // vtblに存在する仮想関数の数
+    int GetVtblNum() const
+    {
+        return vtblNum;
+    }
+    void SetVtblNum( int vtblNum )
+    {
+        this->vtblNum = vtblNum;
+    }
+    void AddVtblNum( int vtblNum )
+    {
+        this->vtblNum += vtblNum;
+    }
+    bool IsExistVirtualFunctions() const
+    {
+        return ( vtblNum > 0 );
+    }
 
     // メンバの総合サイズを取得

BasicCompiler_Common/DebugMiddleFile.cpp

@@ -316 +316 @@
 
         //仮想関数の数
-        *(long *)(buffer+i2)=pobj_c->vtbl_num;
+        *(long *)(buffer+i2)=pobj_c->GetVtblNum();
         i2+=sizeof(long);
 
@@ -698 +698 @@
 
         //仮想関数の数
-        pobj_c->vtbl_num=*(long *)(buffer+i2);
+        pobj_c->SetVtblNum( *(long *)(buffer+i2) );
         i2+=sizeof(long);
 

BasicCompiler_Common/common.h

@@ -47 +47 @@
 
 #ifdef _AMD64_
-#define VER_INFO        "(x64) (rev.275)"
+#define VER_INFO        "(x64) (rev.280)"
 #else
-#define VER_INFO        "(rev.275)"
+#define VER_INFO        "(rev.280)"
 #endif