Kernel/URL.cxx: Remove useless protocols for a kernel.

- remove IteratorInterface for NewFS, design change have been done.
Kernel/NewFS.cxx: Document code.

2 files changed, 40 insertions, 10 deletions

diff --git a/Private/Source/FS/NewFS.cxx b/Private/Source/FS/NewFS.cxx
index ac38400a..012cb3d2 100644
--- a/Private/Source/FS/NewFS.cxx
+++ b/Private/Source/FS/NewFS.cxx
@@ -162,36 +162,69 @@ bool NewFSParser::WriteCatalog(_Input _Output NewCatalog* catalog,
   return false;
 }
 
+/// @brief 
+/// @param catalogName 
+/// @return 
 _Output NewCatalog* NewFSParser::FindCatalog(_Input const char* catalogName) {
   return nullptr;
 }
 
+/// @brief 
+/// @param name 
+/// @return 
 _Output NewCatalog* NewFSParser::GetCatalog(_Input const char* name) {
   return nullptr;
 }
 
+/// @brief 
+/// @param catalog 
+/// @return 
 Boolean NewFSParser::CloseCatalog(_Input _Output NewCatalog* catalog) {
   return false;
 }
 
+/// @brief 
+/// @param catalog 
+/// @return 
 Boolean NewFSParser::RemoveCatalog(_Input _Output NewCatalog* catalog) {
   return false;
 }
 
-
+/// @brief 
+/// @param catalog 
+/// @param dataSz 
+/// @return 
 VoidPtr NewFSParser::ReadCatalog(_Input _Output NewCatalog* catalog,
                                  SizeT dataSz) {
   return nullptr;
 }
 
+/// @brief 
+/// @param catalog 
+/// @param off 
+/// @return 
 bool NewFSParser::Seek(_Input _Output NewCatalog* catalog, SizeT off) {
   return false;
 }
 
+/// @brief 
+/// @param catalog 
+/// @return 
 SizeT NewFSParser::Tell(_Input _Output NewCatalog* catalog) { return 0; }
 
+/// @brief 
+/// @param sz 
+/// @return 
 STATIC Lba ke_find_free_fork(SizeT sz) { return 0; }
+
+/// @brief 
+/// @param sz 
+/// @return 
 STATIC Lba ke_find_free_catalog(SizeT sz) { return 0; }
+
+/// @brief 
+/// @param sz 
+/// @return 
 STATIC Lba ke_find_free_data(SizeT sz) { return 0; }
 
 namespace NewOS::Detail {
diff --git a/Private/Source/URL.cxx b/Private/Source/URL.cxx
index ebb325c1..090c7c66 100644
--- a/Private/Source/URL.cxx
+++ b/Private/Source/URL.cxx
@@ -11,18 +11,15 @@
 /// BUGS: 0
 
 namespace NewOS {
-Url::Url(StringView &strUrl) : fUrlView(strUrl, false) {}
+URL::URL(StringView &strUrl) : fUrlView(strUrl, false) {}
 
-Url::~Url() = default;
+URL::~URL() = default;
 
 constexpr const char *kURLProtocols[] = {
-    "file",   // Filesystem protocol
-    "ping",  // Ping protocol.
-    "telnet", // Telnet protocol
-    "ssh", // SSH protocol
+    "file",  // Filesystem protocol
 };
 
-constexpr const int kUrlOutSz = 3;  //! such as: ://
+constexpr const int kUrlOutSz = 1;  //! such as: ://
 constexpr const int kProtosCount = 4;
 constexpr const int kRangeSz = 4096;
 
@@ -62,7 +59,7 @@ ErrorOr<StringView> url_extract_protocol(const char *url) {
   return view;
 }
 
-Ref<ErrorOr<StringView>> Url::Location() noexcept {
+Ref<ErrorOr<StringView>> URL::Location() noexcept {
   const char *src = fUrlView.Leak().CData();
   auto loc = url_extract_location(src);
 
@@ -71,7 +68,7 @@ Ref<ErrorOr<StringView>> Url::Location() noexcept {
   return Ref<ErrorOr<StringView>>(loc);
 }
 
-Ref<ErrorOr<StringView>> Url::Protocol() noexcept {
+Ref<ErrorOr<StringView>> URL::Protocol() noexcept {
   const char *src = fUrlView.Leak().CData();
   auto loc = url_extract_protocol(src);