Skip to content

Add trivial Sendable conformances #913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 24, 2024
Merged

Add trivial Sendable conformances #913

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions products/llbuildSwift/BuildSystemBindings.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -519,7 +519,7 @@ private final class CommandWrapper {
}

/// Encapsulates a diagnostic as reported by the build system.
public struct Diagnostic {
public struct Diagnostic: Sendable {
public typealias Kind = DiagnosticKind

/// The kind of diagnostic.
Expand Down Expand Up @@ -726,7 +726,7 @@ public struct ProcessHandle: Hashable {
}
}

public struct CommandMetrics {
public struct CommandMetrics: Hashable, Sendable {
public let utime: UInt64 /// User time (in us)
public let stime: UInt64 /// Sys time (in us)
public let maxRSS: UInt64 /// Max RSS (in bytes)
Expand All @@ -739,7 +739,7 @@ public struct CommandMetrics {
}

/// Result of a command execution.
public struct CommandExtendedResult {
public struct CommandExtendedResult: Sendable {
public let result: CommandResult /// The result of a command execution
public let exitStatus: Int32 /// The exit code
public let pid: llbuild_pid_t? /// The process identifier (nil if failed to create a process)
Expand Down
4 changes: 2 additions & 2 deletions products/llbuildSwift/CoreBindings.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ private func stringFromUInt8Array(_ data: [UInt8]) -> String {
}

/// Key objects are used to identify rules that can be built.
public struct Key: CustomStringConvertible, Equatable, Hashable {
public struct Key: CustomStringConvertible, Equatable, Hashable, Sendable {
public let data: [UInt8]

// MARK: CustomStringConvertible Conformance
Expand Down Expand Up @@ -66,7 +66,7 @@ public struct Key: CustomStringConvertible, Equatable, Hashable {
}

/// Value objects are the result of building rules.
public struct Value: CustomStringConvertible, Equatable, Hashable {
public struct Value: CustomStringConvertible, Equatable, Hashable, Sendable {
public let data: [UInt8]

public var description: String {
Expand Down
6 changes: 3 additions & 3 deletions products/llbuildSwift/NinjaManifest.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ public enum NinjaError: Error {
case invalidManifest(errors: String)
}

public struct NinjaRule: Codable, Equatable {
public struct NinjaRule: Codable, Equatable, Sendable {
public let name: String
public let variables: [String: String]

Expand All @@ -28,7 +28,7 @@ public struct NinjaRule: Codable, Equatable {
}
}

public struct NinjaBuildStatement: Codable, Equatable {
public struct NinjaBuildStatement: Codable, Equatable, Sendable {
public let rule: NinjaRule
public let command: String
public let description: String
Expand Down Expand Up @@ -61,7 +61,7 @@ public struct NinjaBuildStatement: Codable, Equatable {
}
}

public struct NinjaManifest: Codable, Equatable {
public struct NinjaManifest: Codable, Equatable, Sendable {
public let rules: [String: NinjaRule]
public let statements: [NinjaBuildStatement]
public let defaultTargets: [String]
Expand Down