diff --git a/docs/develop/storage/Annotations b/docs/develop/storage/Annotations index 3ebcaa2344..6ee3f66746 100644 --- a/docs/develop/storage/Annotations +++ b/docs/develop/storage/Annotations @@ -534,3 +534,9 @@ Location: GetVersionInfo(), SetVersionInfo() Description: Crash when passing a NULL version_info. +Module: BTest +Location: SetName() +Description: Should also rename the mount point. +Priority: medium + + diff --git a/docs/develop/storage/Completed b/docs/develop/storage/Completed index be116ce2b6..ecd5dc1536 100644 --- a/docs/develop/storage/Completed +++ b/docs/develop/storage/Completed @@ -3,3 +3,157 @@ # # This file contains completed ToDo items. +Module: BEntry +Location: SetTo(const BDirectory *, const char *, bool) +Description: Reimplement! Concatenating dir and leaf to an absolute path + prevents the user from accessing entries with longer absolute + path. R5 handles this without problems. +Priority: medium +Requires: - OBOS kernel? + + +Module: BEntry +Location: SetTo(const BDirectory *, const char *, bool) +Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). +Priority: low +Requires: - OBOS kernel + + +Module: BVolume +Location: operator==() +Description: Implement. +Priority: high + + +Module: StatableTest +Location: GetXYZTest() +Description: Uncomment GetVolume() test, when BVolume::==() is implemented. +Priority: low +Requires: - implementation of BVolume::operator==() + + +Module: kernel_interface +Location: set_stat(const char*, StatMember) +Description: Implement WSTAT_CRTIME. +Priority: medium +Comment: not longer necessary as we use the Haiku kernel syscalls now + + +Module: BeOS R5::libroot +Location: +Description: Propose a project wide common handling of the + B_FILE/PATH_NAME_LENGTH (+ 1?) issue. +Priority: low +Comment: B_FILE/PATH_NAME_LENGTH already include the terminating null + + +Module: BSymLink +Location: +Description: Remove the work-around introduced because of the missing FD + version of readlink(). +Priority: medium +Requires: - OBOS kernel + + +Module: BNode +Location: SetTo(const entry_ref *) +Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). +Priority: low +Requires: - OBOS kernel + + +Module: BNode +Location: SetTo(const BEntry *) +Description: Check if necessary to reimplement! Implemented using + SetTo(const entry_ref*). +Priority: low +Requires: - OBOS kernel + + +Module: BNode +Location: SetTo(const BDirectory*, const char*) +Description: Check if necessary to reimplement! Implemented using + SetTo(const BEntry*). +Priority: low +Requires: - OBOS kernel + + +Module: BNode +Location: Lock(), Unlock() +Description: Implement when kernel support is available. +Priority: medium +Requires: - OBOS kernel + + +Module: BFile +Location: SetTo(const entry_ref *, uint32) +Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). +Priority: low +Requires: - OBOS kernel + + +Module: BFile +Location: SetTo(const BEntry *, uint32) +Description: Check if necessary to reimplement! Implemented using + SetTo(const entry_ref*, uint32). +Priority: low +Requires: - OBOS kernel + + +Module: BFile +Location: SetTo(const BDirectory*, const char*, uint32) +Description: Check if necessary to reimplement! Implemented using + SetTo(const BEntry*, uint32). +Priority: low +Requires: - OBOS kernel + + +Module: BDirectory +Location: SetTo(const entry_ref *) +Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). +Priority: low +Requires: - OBOS kernel + + +Module: BDirectory +Location: SetTo(const BEntry *) +Description: Check if necessary to reimplement! Implemented using + SetTo(const entry_ref*). +Priority: low +Requires: - OBOS kernel + + +Module: BDirectory +Location: SetTo(const BDirectory*, const char*) +Description: Check if necessary to reimplement! Implemented using + SetTo(const BEntry*). +Priority: low +Requires: - OBOS kernel + + +Module: BDirectory +Location: GetEntry() +Description: Check if necessary to reimplement! Implemented using + StorageKit::dir_to_self_entry_ref(). +Priority: low +Requires: - OBOS kernel + + +Module: BQuery +Location: SetTarget() +Description: Used a bad hack to get port and token of the BMessenger. Fix it. +Priority: low +Requires: - respective function(s) to be provided by the IK team + + +Module: kernel_interface +Location: remove_attr() +Description: Verify return behavior of fs_remove_attr(). +Priority: medium +Comment: not longer necessary as we use the Haiku kernel syscalls now + + +Module: BVolume +Location: SetName() +Description: Should also rename the mount point. +Priority: medium diff --git a/docs/develop/storage/ToDo b/docs/develop/storage/ToDo index e0910db69f..bb15c84c65 100644 --- a/docs/develop/storage/ToDo +++ b/docs/develop/storage/ToDo @@ -28,22 +28,6 @@ Description: Definition of SYMLINK_MAX belongs to some header file. Remove it. Priority: low -Module: BEntry -Location: SetTo(const BDirectory *, const char *, bool) -Description: Reimplement! Concatenating dir and leaf to an absolute path - prevents the user from accessing entries with longer absolute - path. R5 handles this without problems. -Priority: medium -Requires: - OBOS kernel? - - -Module: BEntry -Location: SetTo(const BDirectory *, const char *, bool) -Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). -Priority: low -Requires: - OBOS kernel - - Module: EntryTest Location: InitTest1(), InitTest2() Description: Enable tests with strlen(dir + leaf) > B_PATH_NAME_LENGTH. @@ -57,82 +41,6 @@ Description: Reimplement for performance reasons. Don't call FlattenedSize(). Priority: low -Module: BVolume -Location: operator==() -Description: Implement. -Priority: high - - -Module: BVolume -Location: SetName() -Description: Should also rename the mount point. -Priority: medium - - -Module: StatableTest -Location: GetXYZTest() -Description: Uncomment GetVolume() test, when BVolume::==() is implemented. -Priority: low -Requires: - implementation of BVolume::operator==() - - -Module: kernel_interface -Location: set_stat(const char*, StatMember) -Description: Implement WSTAT_CRTIME. -Priority: medium - - -Module: kernel_interface -Location: remove_attr() -Description: Verify return behavior of fs_remove_attr(). -Priority: medium - - -Module: BeOS R5::libroot -Location: -Description: Propose a project wide common handling of the - B_FILE/PATH_NAME_LENGTH (+ 1?) issue. -Priority: low - - -Module: BSymLink -Location: -Description: Remove the work-around introduced because of the missing FD - version of readlink(). -Priority: medium -Requires: - OBOS kernel - - -Module: BNode -Location: SetTo(const entry_ref *) -Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). -Priority: low -Requires: - OBOS kernel - - -Module: BNode -Location: SetTo(const BEntry *) -Description: Check if necessary to reimplement! Implemented using - SetTo(const entry_ref*). -Priority: low -Requires: - OBOS kernel - - -Module: BNode -Location: SetTo(const BDirectory*, const char*) -Description: Check if necessary to reimplement! Implemented using - SetTo(const BEntry*). -Priority: low -Requires: - OBOS kernel - - -Module: BNode -Location: Lock(), Unlock() -Description: Implement when kernel support is available. -Priority: medium -Requires: - OBOS kernel - - Module: NodeTest Location: SyncTest() Description: Add more thorough tests. @@ -147,29 +55,6 @@ Priority: medium Requires: - OBOS kernel -Module: BFile -Location: SetTo(const entry_ref *, uint32) -Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). -Priority: low -Requires: - OBOS kernel - - -Module: BFile -Location: SetTo(const BEntry *, uint32) -Description: Check if necessary to reimplement! Implemented using - SetTo(const entry_ref*, uint32). -Priority: low -Requires: - OBOS kernel - - -Module: BFile -Location: SetTo(const BDirectory*, const char*, uint32) -Description: Check if necessary to reimplement! Implemented using - SetTo(const BEntry*, uint32). -Priority: low -Requires: - OBOS kernel - - Module: BFile Location: Read/Write[At]() Description: Verify behavior of B_OPEN_AT_END. @@ -183,37 +68,6 @@ Priority: medium Requires: - verification of B_OPEN_AT_END behavior -Module: BDirectory -Location: SetTo(const entry_ref *) -Description: Reimplement! Implemented using StorageKit::entry_ref_to_path(). -Priority: low -Requires: - OBOS kernel - - -Module: BDirectory -Location: SetTo(const BEntry *) -Description: Check if necessary to reimplement! Implemented using - SetTo(const entry_ref*). -Priority: low -Requires: - OBOS kernel - - -Module: BDirectory -Location: SetTo(const BDirectory*, const char*) -Description: Check if necessary to reimplement! Implemented using - SetTo(const BEntry*). -Priority: low -Requires: - OBOS kernel - - -Module: BDirectory -Location: GetEntry() -Description: Check if necessary to reimplement! Implemented using - StorageKit::dir_to_self_entry_ref(). -Priority: low -Requires: - OBOS kernel - - Module: fs_info.h Location: struct fs_info Description: Change "char device_name[128]" to "char device_name[B_DEV_NAME_LENGTH]" @@ -223,13 +77,6 @@ Priority: low Requires: - filesystem support files in the source hierarchy -Module: BQuery -Location: SetTarget() -Description: Used a bad hack to get port and token of the BMessenger. Fix it. -Priority: low -Requires: - respective function(s) to be provided by the IK team - - Module: BMimeType/MimeTypeTest Location: update_mime_info(), create_app_meta_mime() Description: Find out, what is the meaning of the force parameter. It does @@ -241,3 +88,9 @@ Priority: medium Requires: +Module: VolumeTest +Location: SetNameTest() +Description: Check whether the mount point is renamed properly. +Priority: low + +