Skip to content

Commit

Permalink
Merge pull request #1205 from muzarski/column-type-non-exhaustive
Browse files Browse the repository at this point in the history
result: mark ColumnType and related enums as non-exhaustive
  • Loading branch information
wprzytula authored Feb 25, 2025
2 parents d190409 + 25179ab commit fc694ec
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions scylla-cql/src/frame/response/result.rs
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ pub struct TableSpec<'a> {
/// types those fields will always be set to `false` (even if the DB column
/// corresponding to given marker / result type is frozen).
#[derive(Clone, Debug, PartialEq, Eq)]
#[non_exhaustive]
pub enum ColumnType<'frame> {
/// Types that are "simple" (non-recursive).
Native(NativeType),
Expand Down Expand Up @@ -82,6 +83,7 @@ pub enum ColumnType<'frame> {

/// A [ColumnType] variants that are "simple" (non-recursive).
#[derive(Clone, Debug, PartialEq, Eq)]
#[non_exhaustive]
pub enum NativeType {
Ascii,
Boolean,
Expand Down Expand Up @@ -110,6 +112,7 @@ pub enum NativeType {
///
/// Tuple and vector are not collections because they have predefined size.
#[derive(Clone, Debug, PartialEq, Eq)]
#[non_exhaustive]
pub enum CollectionType<'frame> {
List(Box<ColumnType<'frame>>),
Map(Box<ColumnType<'frame>>, Box<ColumnType<'frame>>),
Expand Down

0 comments on commit fc694ec

Please sign in to comment.