#
d7857201 |
| 28-Jan-2025 |
Ed Tanous <etanous@nvidia.com> |
Fix includes
Clang-tidy misc-include-cleaner appears to now be enforcing significantly more headers than previously. That is overall a good thing, but forces us to fix some issues. This commit is
Fix includes
Clang-tidy misc-include-cleaner appears to now be enforcing significantly more headers than previously. That is overall a good thing, but forces us to fix some issues. This commit is largely just taking the clang-recommended fixes and checking them in. Subsequent patches will fix the more unique issues.
Note, that a number of new ignores are added into the .clang-tidy file. These can be cleaned up over time as they're understood. The majority are places where boost includes a impl/x.hpp and x.hpp, but expects you to use the later. include-cleaner opts for the impl, but it isn't clear why.
Change-Id: Id3fdd7ee6df6c33b2fd35626898523048dd51bfb Signed-off-by: Ed Tanous <etanous@nvidia.com> Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
show more ...
|
#
40e9b92e |
| 10-Sep-2024 |
Ed Tanous <etanous@nvidia.com> |
Use SPDX identifiers
SPDX identifiers are simpler, and reduce the amount of cruft we have in code files. They are recommended by linux foundation, and therefore we should do as they allow.
This pa
Use SPDX identifiers
SPDX identifiers are simpler, and reduce the amount of cruft we have in code files. They are recommended by linux foundation, and therefore we should do as they allow.
This patchset does not intend to modify any intent on any existing copyrights or licenses, only to standardize their inclusion.
[1] https://www.linuxfoundation.org/blog/blog/copyright-notices-in-open-source-software-projects
Change-Id: I935c7c0156caa78fc368c929cebd0f068031e830 Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
#
9ea15c35 |
| 04-Jan-2022 |
Ed Tanous <edtanous@google.com> |
Include what you use
Do a partial update from the include what you use tool.
While ideally we'd be able to do this as part of CI, there's still quite a bit of noise in the output that requires manu
Include what you use
Do a partial update from the include what you use tool.
While ideally we'd be able to do this as part of CI, there's still quite a bit of noise in the output that requires manual intervention.
Tested: Code compiles
Signed-off-by: Ed Tanous <edtanous@google.com> Change-Id: Iaaeb7a9199f64b5d6913c3abab4779b252768ed8
show more ...
|
#
287ece64 |
| 30-Sep-2021 |
George Liu <liuxiwei@inspur.com> |
Remove NTPServers duplicate values and null values
When saving the set NTPServers values from webUI, NTPServer may contain duplicate values and null values and update them to D-Bus.
Now, need to pa
Remove NTPServers duplicate values and null values
When saving the set NTPServers values from webUI, NTPServer may contain duplicate values and null values and update them to D-Bus.
Now, need to parse and verify the value of the ntpServers attribute,and remove duplicate values and null values.
Tested:save NTP and check it via D-Bus without this patch: NTPServers property as 3 "" "10.164.29.2" "10.164.29.2"
with this patch: NTPServers property as 2 "" "10.164.29.2"
Signed-off-by: George Liu <liuxiwei@inspur.com> Change-Id: I52291e4608efd635b179f3934c3d3e805afd2209
show more ...
|