Skip to content
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

Jump to bottom

Row and column slice getter #55

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
362 −21
Open

Row and column slice getter #55

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift click to select a range
5d33f75
Quick linter fixes
ibarrond Aug 22, 2024
c2b2ad7
get_row and get_col
ibarrond Aug 22, 2024
a22eaa9
iter_rows_as_slices and iter_cols_as_slices
ibarrond Aug 28, 2024
8ab91df
IntoIterator
ibarrond Aug 28, 2024
32f6ae1
Avoiding anonymous types in as_slice iterators
ibarrond Aug 28, 2024
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
Next Next commit
Quick linter fixes
  • Loading branch information
@ibarrond
ibarrond committed Aug 22, 2024
commit 5d33f75c05b212c7f2f78ad55dcd2aa11eae37fb
44 changes: 23 additions & 21 deletions src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,15 52,15 @@ use serde::{
ser::{Serialize, SerializeStruct, Serializer},
};

use core::cmp::Eq;
use core::fmt;
use core::hash;
use core::iter::StepBy;
use core::ops::Index;
use core::ops::IndexMut;
use core::slice::Iter;
use core::slice::IterMut;
use core::{cmp, convert::TryInto};
use core::{
cmp,
cmp::Eq,
convert::TryInto,
fmt, hash,
iter::StepBy,
ops::{Index, IndexMut},
slice::{Iter, IterMut},
};

#[doc(hidden)]
#[macro_export]
Expand Down Expand Up @@ -321,7 321,7 @@ impl<T> Grid<T> {
///
/// ```
/// use grid::Grid;
/// let grid : Grid<u8> = Grid::new(2,3);
/// let grid: Grid<u8> = Grid::new(2, 3);
/// assert_eq!(grid[(0, 0)], 0);
/// ```
///
Expand Down Expand Up @@ -422,7 422,7 @@ impl<T> Grid<T> {
///
/// ```
/// use grid::Grid;
/// let grid = Grid::from_vec(vec![1,2,3,4,5,6], 3);
/// let grid = Grid::from_vec(vec![1, 2, 3, 4, 5, 6], 3);
/// assert_eq!(grid.size(), (2, 3));
/// ```
///
Expand All @@ -434,7 434,7 @@ impl<T> Grid<T> {
///
/// ``` should_panic
/// use grid::Grid;
/// Grid::from_vec(vec![1,2,3,4,5], 3);
/// Grid::from_vec(vec![1, 2, 3, 4, 5], 3);
/// ```
///
/// # Panics
Expand Down Expand Up @@ -493,7 493,8 @@ impl<T> Grid<T> {
///
/// # Safety
///
/// Calling this method with an out-of-bounds index is undefined behavior even if the resulting reference is not used.
/// Calling this method with an out-of-bounds index is undefined behavior even
/// if the resulting reference is not used.
#[inline]
#[must_use]
pub unsafe fn get_unchecked(&self, row: impl Into<usize>, col: impl Into<usize>) -> &T {
Expand All @@ -506,7 507,8 @@ impl<T> Grid<T> {
///
/// # Safety
///
/// Calling this method with an out-of-bounds index is undefined behavior even if the resulting reference is not used.
/// Calling this method with an out-of-bounds index is undefined behavior even
/// if the resulting reference is not used.
#[inline]
#[must_use]
pub unsafe fn get_unchecked_mut(
Expand Down Expand Up @@ -577,7 579,7 @@ impl<T> Grid<T> {
/// For example:
/// ```
/// use grid::*;
/// let grid : Grid<u8> = grid![];
/// let grid: Grid<u8> = grid![];
/// assert!(grid.is_empty());
/// ```
#[must_use]
Expand Down Expand Up @@ -875,7 877,7 @@ impl<T> Grid<T> {
/// ```
/// use grid::*;
/// let mut grid: Grid<u8> = grid![];
/// let row = vec![1,2,3];
/// let row = vec![1, 2, 3];
/// grid.push_row(row);
/// assert_eq!(grid.size(), (1, 3));
/// ```
Expand Down Expand Up @@ -929,7 931,7 @@ impl<T> Grid<T> {
/// ```
/// use grid::*;
/// let mut grid: Grid<u8> = grid![];
/// let col = vec![1,2,3];
/// let col = vec![1, 2, 3];
/// grid.push_col(col);
/// assert_eq!(grid.size(), (3, 1));
/// ```
Expand Down Expand Up @@ -1336,8 1338,8 @@ impl<T> Grid<T> {
///
/// # Performance
///
/// This method will be significantly slower if the grid initialy uses a column-major memory layout,
/// which is the default.
/// This method will be significantly slower if the grid initialy uses a
/// column-major memory layout, which is the default.
pub fn rotate_left(&mut self) {
self.transpose();
self.flip_rows();
Expand All @@ -1358,8 1360,8 @@ impl<T> Grid<T> {
///
/// # Performance
///
/// This method will be significantly slower if the grid initialy uses a row-major memory layout,
/// which is the default.
/// This method will be significantly slower if the grid initialy uses a
/// row-major memory layout, which is the default.
pub fn rotate_right(&mut self) {
self.transpose();
self.flip_cols();
Expand Down