Skip to content

Commit

Permalink
Fix spelling
Browse files Browse the repository at this point in the history
  • Loading branch information
PatrickKa committed Dec 22, 2023
1 parent 2f51c29 commit 2ff78b8
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Sts1CobcSw/Hal/Communication.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
// TODO: Add declarations at the top to see all provided functionality at once
namespace sts1cobcsw::hal
{
// TODO: Remove extent to avoid code bload?
// TODO: Remove extent to avoid code bloat?
template<typename T, std::size_t extent>
auto WriteTo(auto * communicationInterface, std::span<T const, extent> data) -> void;

Expand Down
2 changes: 1 addition & 1 deletion Sts1CobcSw/Utility/Span.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
//! in this file to turn single objects or arrays of objects to the "correct" `std::span`.
//! Parameters passed by `& const` are converted to spans with read-only elements, and those passed
//! as a non-const pointer are converted to spans with mutable elements. This solves the problem
//! that `std::span<T>` does not implicitely convert to `std::span<T const>`.
//! that `std::span<T>` does not implicitly convert to `std::span<T const>`.

#pragma once

Expand Down
2 changes: 1 addition & 1 deletion Tests/UnitTests/Span.test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
using sts1cobcsw::Span;


// The extra level of indirection privided by this function object is necessary to test if some
// The extra level of indirection provided by this function object is necessary to test if some
// calls to Span() do not compile without actually throwing compiler errors. The magic lies in
// decltype(expression) which only pretends to evaluate the expression to get its type. If the
// expression is ill-formed (i.e. it would not compile) the operator() for this type T is removed
Expand Down

0 comments on commit 2ff78b8

Please sign in to comment.