Phase 122: Complete LuaT enum class refactoring

Fixed all remaining type mismatches from lu_byte to LuaT:
- Updated operator new signatures in all GC classes to use LuaT
- Changed luaC_newobj/luaC_newobjdt to accept LuaT instead of lu_byte
- Updated Table method return types (get, getInt, getShortStr, getStr)
- Fixed local variables in ltable.cpp, ldump.cpp, lstate.cpp
- Added iscollectable(LuaT) overload for type-safe bit checking
- Updated macros in gc_marking.cpp and gc_weak.cpp to use iscollectable()
- Changed createstrobj parameter from lu_byte to LuaT

All files now use LuaT enum class instead of lu_byte for variant tags.
Build successful with zero warnings!

Author: claude

src/core/lstate.cpp

@@ -272,7 +272,7 @@ LUA_API lua_State *lua_newthread (lua_State *L) {
   lua_lock(L);
   luaC_checkGC(L);
   /* create new thread */
-  o = luaC_newobjdt(L, LUA_TTHREAD, sizeof(LX), lxOffset());
+  o = luaC_newobjdt(L, LuaT::THREAD, sizeof(LX), lxOffset());
   L1 = gco2th(o);
   /* anchor it on L stack */
   setthvalue2s(L, L->getTop().p, L1);

src/memory/gc/gc_marking.cpp

@@ -83,7 +83,7 @@ static inline void linkgclistThread(lua_State* th, GCObject*& p) {
 ** Access to collectable objects in table array part
 */
 #define gcvalarr(t, i)  \
-    ((*(t)->getArrayTag(i) & BIT_ISCOLLECTABLE) ? (t)->getArrayVal(i)->gc : nullptr)
+    (iscollectable(*(t)->getArrayTag(i)) ? (t)->getArrayVal(i)->gc : nullptr)
 
 
 /*

src/memory/gc/gc_weak.cpp

@@ -38,7 +38,7 @@
 
 /* Access to collectable objects in array part of tables */
 #define gcvalarr(t,i)  \
-	((*(t)->getArrayTag(i) & BIT_ISCOLLECTABLE) ? (t)->getArrayVal(i)->gc : nullptr)
+	(iscollectable(*(t)->getArrayTag(i)) ? (t)->getArrayVal(i)->gc : nullptr)
 
 /* Note: gcvalueN and valiswhite are now in lgc.h */
 /* Note: markkey and markvalue are defined in gc_marking.h */

src/memory/lgc.cpp

@@ -219,7 +219,7 @@ void luaC_barrierback_ (lua_State *L, GCObject *o) {
 ** create a new collectable object (with given type, size, and offset)
 ** and link it to 'allgc' list.
 */
-GCObject *luaC_newobjdt (lua_State *L, lu_byte tt, size_t sz, size_t offset) {
+GCObject *luaC_newobjdt (lua_State *L, LuaT tt, size_t sz, size_t offset) {
   global_State *g = G(L);
   char *p = cast_charp(luaM_newobject(L, novariant(tt), sz));
   GCObject *o = reinterpret_cast<GCObject*>(p + offset);
@@ -234,7 +234,7 @@ GCObject *luaC_newobjdt (lua_State *L, lu_byte tt, size_t sz, size_t offset) {
 /*
 ** create a new collectable object with no offset.
 */
-GCObject *luaC_newobj (lua_State *L, lu_byte tt, size_t sz) {
+GCObject *luaC_newobj (lua_State *L, LuaT tt, size_t sz) {
   return luaC_newobjdt(L, tt, sz, 0);
 }
 

src/memory/lgc.h

@@ -387,8 +387,8 @@ LUAI_FUNC void luaC_step (lua_State *L);
 LUAI_FUNC void luaC_runtilstate (lua_State *L, GCState state, int fast);
 LUAI_FUNC void luaC_fullgc (lua_State *L, int isemergency);
 LUAI_FUNC void propagateall (global_State *g);  /* used by GCCollector */
-LUAI_FUNC GCObject *luaC_newobj (lua_State *L, lu_byte tt, size_t sz);
-LUAI_FUNC GCObject *luaC_newobjdt (lua_State *L, lu_byte tt, size_t sz,
+LUAI_FUNC GCObject *luaC_newobj (lua_State *L, LuaT tt, size_t sz);
+LUAI_FUNC GCObject *luaC_newobjdt (lua_State *L, LuaT tt, size_t sz,
                                                  size_t offset);
 LUAI_FUNC void luaC_barrier_ (lua_State *L, GCObject *o, GCObject *v);
 LUAI_FUNC void luaC_barrierback_ (lua_State *L, GCObject *o);
@@ -411,37 +411,37 @@ LUAI_FUNC void freeobj (lua_State *L, GCObject *o);
 */
 
 // CClosure placement new operator
-inline void* CClosure::operator new(size_t size, lua_State* L, lu_byte tt, size_t extra) {
+inline void* CClosure::operator new(size_t size, lua_State* L, LuaT tt, size_t extra) {
   return luaC_newobj(L, tt, size + extra);
 }
 
 // LClosure placement new operator
-inline void* LClosure::operator new(size_t size, lua_State* L, lu_byte tt, size_t extra) {
+inline void* LClosure::operator new(size_t size, lua_State* L, LuaT tt, size_t extra) {
   return luaC_newobj(L, tt, size + extra);
 }
 
 // Udata placement new operator
-inline void* Udata::operator new(size_t size, lua_State* L, lu_byte tt, size_t extra) {
+inline void* Udata::operator new(size_t size, lua_State* L, LuaT tt, size_t extra) {
   return luaC_newobj(L, tt, size + extra);
 }
 
 // TString placement new operator
-inline void* TString::operator new(size_t size, lua_State* L, lu_byte tt, size_t extra) {
+inline void* TString::operator new(size_t size, lua_State* L, LuaT tt, size_t extra) {
   return luaC_newobj(L, tt, size + extra);
 }
 
 // Proto placement new operator
-inline void* Proto::operator new(size_t size, lua_State* L, lu_byte tt) {
+inline void* Proto::operator new(size_t size, lua_State* L, LuaT tt) {
   return luaC_newobj(L, tt, size);
 }
 
 // UpVal placement new operator
-inline void* UpVal::operator new(size_t size, lua_State* L, lu_byte tt) {
+inline void* UpVal::operator new(size_t size, lua_State* L, LuaT tt) {
   return luaC_newobj(L, tt, size);
 }
 
 // Table placement new operator
-inline void* Table::operator new(size_t size, lua_State* L, lu_byte tt) {
+inline void* Table::operator new(size_t size, lua_State* L, LuaT tt) {
   return luaC_newobj(L, tt, size);
 }
 

src/objects/lfunc.h

@@ -87,7 +87,7 @@ class UpVal : public GCBase<UpVal> {
   ~UpVal() noexcept = default;
 
   // Phase 50: Placement new operator - integrates with Lua's GC (implemented in lgc.h)
-  static void* operator new(size_t size, lua_State* L, lu_byte tt);
+  static void* operator new(size_t size, lua_State* L, LuaT tt);
 
   // Disable regular new/delete (must use placement new with GC)
   static void* operator new(size_t) = delete;
@@ -144,7 +144,7 @@ class CClosure : public GCBase<CClosure> {
 
 public:
   // Member placement new operator for GC allocation (defined in lgc.h)
-  static void* operator new(size_t size, lua_State* L, lu_byte tt, size_t extra = 0);
+  static void* operator new(size_t size, lua_State* L, LuaT tt, size_t extra = 0);
 
   // Constructor
   CClosure(int nupvals);
@@ -182,7 +182,7 @@ class LClosure : public GCBase<LClosure> {
 
 public:
   // Member placement new operator for GC allocation (defined in lgc.h)
-  static void* operator new(size_t size, lua_State* L, lu_byte tt, size_t extra = 0);
+  static void* operator new(size_t size, lua_State* L, LuaT tt, size_t extra = 0);
 
   // Constructor
   LClosure(int nupvals);

src/objects/lobject_core.h

@@ -302,6 +302,7 @@ class GCBase: public GCObject {
 };
 
 constexpr bool iscollectable(const TValue* o) noexcept { return (static_cast<int>(rawtt(o)) & BIT_ISCOLLECTABLE) != 0; }
+constexpr bool iscollectable(LuaT tag) noexcept { return (static_cast<int>(tag) & BIT_ISCOLLECTABLE) != 0; }
 
 constexpr bool TValue::isCollectable() const noexcept { return (static_cast<int>(tt_) & BIT_ISCOLLECTABLE) != 0; }
 
@@ -381,7 +382,7 @@ class Udata : public GCBase<Udata> {
   }
 
   // Phase 50: Placement new operator - integrates with Lua's GC (implemented in lgc.h)
-  static void* operator new(size_t size, lua_State* L, lu_byte tt, size_t extra = 0);
+  static void* operator new(size_t size, lua_State* L, LuaT tt, size_t extra = 0);
 
   // Disable regular new/delete (must use placement new with GC)
   static void* operator new(size_t) = delete;

src/objects/lproto.h

@@ -248,7 +248,7 @@ class Proto : public GCBase<Proto> {
   ~Proto() noexcept = default;
 
   // Phase 50: Placement new operator - integrates with Lua's GC (implemented in lgc.h)
-  static void* operator new(size_t size, lua_State* L, lu_byte tt);
+  static void* operator new(size_t size, lua_State* L, LuaT tt);
 
   // Disable regular new/delete (must use placement new with GC)
   static void* operator new(size_t) = delete;

src/objects/lstring.cpp

@@ -146,7 +146,7 @@ void TString::init(lua_State* L) {
 ** creates a new string object
 ** Phase 50: Now uses placement new to call constructor
 */
-static TString *createstrobj (lua_State *L, size_t totalsize, lu_byte tag,
+static TString *createstrobj (lua_State *L, size_t totalsize, LuaT tag,
                               unsigned h) {
   // For TString, we need to allocate exactly totalsize bytes, not sizeof(TString)
   // For short strings: totalsize = contentsOffset() + string_length + 1

src/objects/lstring.h

@@ -99,7 +99,7 @@ class TString : public GCBase<TString> {
 
   // Phase 50: Placement new operator - integrates with Lua's GC (implemented in lgc.h)
   // Note: For TString, this may allocate less than sizeof(TString) for short strings!
-  static void* operator new(size_t size, lua_State* L, lu_byte tt, size_t extra = 0);
+  static void* operator new(size_t size, lua_State* L, LuaT tt, size_t extra = 0);
 
   // Disable regular new/delete (must use placement new with GC)
   static void* operator new(size_t) = delete;