Kernel: Remove the now-unused FS::DirectoryEntry

This object was cumbersome and annoying (mostly due to its manually
managed, statically sized name buffer.) And now we no longer need it!

Kernel/FileSystem/FileSystem.cpp

@@ -65,26 +65,6 @@ FS* FS::from_fsid(u32 id)
     return nullptr;
 }
 
-FS::DirectoryEntry::DirectoryEntry(const char* n, InodeIdentifier i, u8 ft)
-    : name_length(strlen(n))
-    , inode(i)
-    , file_type(ft)
-{
-    ASSERT(name_length < (int)sizeof(name));
-    memcpy(name, n, name_length);
-    name[name_length] = '\0';
-}
-
-FS::DirectoryEntry::DirectoryEntry(const char* n, size_t nl, InodeIdentifier i, u8 ft)
-    : name_length(nl)
-    , inode(i)
-    , file_type(ft)
-{
-    ASSERT(name_length < (int)sizeof(name));
-    memcpy(name, n, nl);
-    name[nl] = '\0';
-}
-
 FS::DirectoryEntryView::DirectoryEntryView(const StringView& n, InodeIdentifier i, u8 ft)
     : name(n)
     , inode(i)

Kernel/FileSystem/FileSystem.h

@@ -69,16 +69,6 @@ public:
 
     virtual KResult prepare_to_unmount() const { return KSuccess; }
 
-    // FIXME: This data structure is very clunky and unpleasant. Replace it with something nicer.
-    struct DirectoryEntry {
-        DirectoryEntry(const char* name, InodeIdentifier, u8 file_type);
-        DirectoryEntry(const char* name, size_t name_length, InodeIdentifier, u8 file_type);
-        char name[256];
-        size_t name_length { 0 };
-        InodeIdentifier inode;
-        u8 file_type { 0 };
-    };
-
     struct DirectoryEntryView {
         DirectoryEntryView(const StringView& name, InodeIdentifier, u8 file_type);