This repository was archived by the owner on May 28, 2025. It is now read-only.
Finish NSFileHandle's stubbed methods#2477
Merged
DHowett-MSFT merged 2 commits intoApr 11, 2017
Merged
Conversation
This commit marks roughly half of NSFileHandle's APIs NotInPlan. WinSock deals in opaque handles which cannot be used with open/close/seek/tell/read/write, so most of NSFileHandle's APIs cannot be supported with the POSIX File Descriptor interface. The background APIs are better suited to use with a socket than with a file, so until somebody asks for socket support, we will mark them unplanned. File handles cannot be coded except through IPC coders, but WinObjC does not have any IPC coders. Fixes microsoft#2367.
DHowett-MSFT
commented
Apr 10, 2017
| @Status Interoperable | ||
| */ | ||
| + (NSFileHandle*)fileHandleWithNullDevice { | ||
| return [[_NSFileHandleNullDevice new] autorelease]; |
Author
There was a problem hiding this comment.
TakeOwnership LIIIIIIVESSSSSSSSS
aballway
approved these changes
Apr 10, 2017
| @Notes NSFileHandle only supports file URLs. | ||
| */ | ||
| + (instancetype)fileHandleForReadingFromURL:(NSURL*)url error:(NSError* _Nullable*)error { | ||
| return [NSFileHandle _fileHandleForURl:url error:error openType:_NSFileOpenModeRead]; |
| - (instancetype)_initWithFileAtPath:(NSString*)file openType:(_NSFileOpenMode)type { | ||
| if (file == nil) { | ||
| [self release]; | ||
| return nil; |
Contributor
There was a problem hiding this comment.
nit: empty space after curly brace.
| O_RDWR | _O_CREAT | O_APPEND | _O_BINARY, | ||
| _SH_DENYNO, | ||
| _S_IREAD | _S_IWRITE); | ||
| default: |
Contributor
There was a problem hiding this comment.
Since this is internal, should we fail if we get an invalid type?
| - (void)waitForDataInBackgroundAndNotifyForModes:(NSArray*)modes STUB_METHOD; | ||
| @property (readonly) int fileDescriptor; | ||
| @property (readonly, copy) NSData* availableData; | ||
| @property (readonly) unsigned long long offsetInFile; |
Contributor
There was a problem hiding this comment.
super duper ultra nit: no implied int
rajsesh
suggested changes
Apr 10, 2017
| NSString* const NSFileHandleNotificationDataItem = @"NSFileHandleNotificationDataItem"; | ||
| NSString* const NSFileHandleOperationException = @"NSFileHandleOperationException"; | ||
| NSString* const NSFileHandleNotificationMonitorModes = @"NSFileHandleNotificationMonitorModes"; | ||
| NSString* const NSFileHandleConnectionAcceptedNotification = @"NSFileHandleConnectionAccepted"; |
Contributor
There was a problem hiding this comment.
Do we want to "fix" these strings if they are not the right values already?
|
|
||
| /** | ||
| @Status Stub | ||
| @Notes |
Contributor
There was a problem hiding this comment.
The header file must be updated too, to remove the STUB_* macros (and add NOTINPLAN_* macros where applicable.
Contributor
There was a problem hiding this comment.
Ugh I can't see things today, my bad :(
rajsesh
approved these changes
Apr 10, 2017
|
|
||
| /** | ||
| @Status Stub | ||
| @Notes |
Contributor
There was a problem hiding this comment.
Ugh I can't see things today, my bad :(
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit marks roughly half of NSFileHandle's APIs NotInPlan.
WinSock deals in opaque handles which cannot be used with
open/close/seek/tell/read/write, so most of NSFileHandle's APIs cannot
be supported with the POSIX File Descriptor interface.
The background APIs are better suited to use with a socket than with a
file, so until somebody asks for socket support, we will mark them
unplanned.
File handles cannot be coded except through IPC coders, but WinObjC does
not have any IPC coders.
Fixes #2367.