Possible to speed up some algorithms
Details
Details
Diff Detail
Diff Detail
- Repository
- R262 LabPlot
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
Thanks in advance for fixing minor documentation issues.
| src/backend/core/AbstractColumn.cpp | ||
|---|---|---|
| 545 ↗ | (On Diff #49571) | Doxygen's \brief is for classes, structures, etc. This line is redundant. It is sufficient to begin with the next line. |
| 548 ↗ | (On Diff #49571) | No need to write \return if there will be no description of the returned value. |
| src/backend/core/column/Column.cpp | ||
| 482 ↗ | (On Diff #49571) | For Doxygen to work, this should be \see AbstractColumn::properties |
| 483 ↗ | (On Diff #49571) | No need to write \return if there will be no description of the returned value. |
| src/backend/core/column/ColumnPrivate.cpp | ||
| 1216 ↗ | (On Diff #49571) | Doxygen's \brief is for classes, structures, etc. This line is redundant. It is sufficient to begin with the next line. |
| src/backend/core/AbstractColumn.cpp | ||
|---|---|---|
| 547 ↗ | (On Diff #49571) | We'll need to change this "will be used" to "is used" later. |
| src/backend/core/column/ColumnPrivate.cpp | ||
| 1311 ↗ | (On Diff #49571) | can we have this return already in the if-else block before this for-loop? Then we don't need to have this check inside the loop. Also, we should maybe set propertiesAvailable to true if we return for unsupported data types so we don't call this function again and again. |
| src/backend/core/column/ColumnPrivate.h | ||
| 117 ↗ | (On Diff #49571) | int? Why not Properties? |