i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Squashed several TODO items by changing the Storage Kit <-> Kernel interface.

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/trunk/current@8691 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Ingo Weinhold
2004-08-28 20:04:41 +00:00
parent 121af366ad
commit eb98f8c339
3 changed files with 166 additions and 153 deletions
Download Patch File Download Diff File Expand all files Collapse all files
docs/develop/storage/Annotations
+6
View File
@@ -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
docs/develop/storage/Completed
+154
View File
@@ -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
docs/develop/storage/ToDo
+6 -153
View File
@@ -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