Page MenuHomeFreeBSD
Differential D15383

Fix build for platforms using GCC
ClosedPublic
Actions

Authored by lwhsu on May 10 2018, 4:51 PM.
Tags
None
Referenced Files
F103215824: D15383.id42372.diff
Fri, Nov 22, 7:53 AM
Unknown Object (File)
Mon, Nov 18, 2:59 AM
Unknown Object (File)
Mon, Nov 18, 2:54 AM
Unknown Object (File)
Sun, Nov 17, 10:06 PM
Unknown Object (File)
Sep 17 2024, 10:43 AM
Unknown Object (File)
Sep 7 2024, 6:28 PM
Unknown Object (File)
Sep 5 2024, 3:05 AM
Unknown Object (File)
Jul 9 2024, 11:26 PM
View All 22 Files
Subscribers
imp
jhibbits

Details

Reviewers
mav
jhibbits
Group Reviewers
cam
Commits
rS360687: Redo r360682, now with only a minimal fix for misleading indentation:
rS360686: Revert r360682, as it does not compile on stable/11 (will commit a
rS360682: MFC r333465 (by lwhsu):
rS333465: Fix build for platforms using GCC:
Summary

Fix build for platforms using GCC:

  • Remove unused or dead store variable
  • Remove unused function ctl_copyin_alloc
  • Add missing curly brackets, this seems a regression in r287720

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lwhsu created this revision.May 10 2018, 4:51 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 10 2018, 4:51 PM
Harbormaster completed remote builds in B16518: Diff 42370.May 10 2018, 4:51 PM
lwhsu added inline comments.May 10 2018, 4:54 PM
sys/cam/ctl/ctl.c
8610 ↗(On Diff #42370)

I'm not 100% sure about this is right, but I highly suspect it should do so after checking https://svnweb.freebsd.org/base/head/sys/cam/ctl/ctl.c?annotate=333446#l8481

jhibbits accepted this revision.May 10 2018, 5:05 PM
jhibbits added a subscriber: jhibbits.

If it builds, looks good to me.

This revision is now accepted and ready to land.May 10 2018, 5:05 PM
Closed by commit rS333465: Fix build for platforms using GCC: (authored by lwhsu). · Explain WhyMay 10 2018, 5:22 PM
This revision was automatically updated to reflect the committed changes.
dim added a commit: rS360682: MFC r333465 (by lwhsu):.May 6 2020, 5:33 AM
Herald added a reviewer: cam. · View Herald TranscriptMay 6 2020, 5:33 AM
dim added a commit: rS360686: Revert r360682, as it does not compile on stable/11 (will commit a.May 6 2020, 8:24 AM
dim added a commit: rS360687: Redo r360682, now with only a minimal fix for misleading indentation:.May 6 2020, 8:33 AM

Revision Contents
Changeset List

PathSize
head/
sys/
cam/
ctl/
37 lines

Diff 42372

View Options

head/sys/cam/ctl/ctl.c

Loading...