chore(cmake): remove unnecessary whitespace patch #1522
Conversation
Signed-off-by: Leonardo Grasso <me@leonardograsso.com>
| - if(udig_alloc_ring_descriptors(&(handle->m_devs[0].m_bufinfo_fd), | ||
| - &handle->m_devs[0].m_bufinfo, | ||
| + if(udig_alloc_ring_descriptors(&(handle->m_devs[0].m_bufinfo_fd), | ||
| + &handle->m_devs[0].m_bufinfo, |
There was a problem hiding this comment.
N.B.:
this hunk ☝️ was just removing a trailing white space at ...&handle->m_devs[0].m_bufinfo, . Not actually needed.
|
LGTM label has been added. Git tree hash: 152678348027cc94aca865bd54afaf2b97f22b4a
|
leogr
changed the title
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fntlnz, leodido The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
1 similar comment
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fntlnz, leodido The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/milestone 0.27.0 |
What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
Very small improvement required to reduce the likelihood that the patching process will fail when the upstream file gets modified.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Not a big issue right now, but may be useful in development since the patching process is already failing if using the latest commit from sysdig. I stumbled upon this problem while experimenting.
/cc @leodido
Does this PR introduce a user-facing change?: