Skip to content

Removed use of unsafe "sprintf"

Mohsen Lahooti requested to merge feature/nonconformal-MR-p2 into master

Issue/feature addressed

sprintf is inherently unsafe, as it does not check for buffer overflows.

Proposed solution

Replace the use of sprintf with a more modern and safer approach.

Implementation

Replaced deprecated sprintf with std::to_string

Tests

None

Notes

Checklist

  • Functions and classes, or changes to them, are documented.
  • User guide/documentation is updated.
  • Changelog is updated.
  • Suitable tests added for new functionality.
  • Contributed code is correctly formatted. (See the contributing guidelines).
  • License added to any new files.
  • No extraneous files have been added (e.g. compiler output or test data files).

Warning

On the 19.07 the code formatting (code style) was standardised using clang-format, over the whole Nektar++ code. This means changes in your branch will conflict with formatting changes on the master branch. To resolve these conflicts , see #295 (closed)

Edited by Chris Cantwell

Merge request reports