Differential D36735

libsa/arp.c: Change printf format string
ClosedPublic
Actions

Authored by mig_semihalf.com on Sep 26 2022, 11:02 PM.

Tags

None

Referenced Files

	F85204378: D36735.diff
	Mon, Jun 3, 1:08 AM

	Unknown Object (File)
	Tue, May 21, 12:33 PM

	Unknown Object (File)
	Tue, May 21, 12:32 PM

	Unknown Object (File)
	Tue, May 21, 12:25 PM

	Unknown Object (File)
	Tue, May 21, 5:31 AM

	Unknown Object (File)
	Sun, May 19, 2:58 AM

	Unknown Object (File)
	Sat, May 18, 6:00 PM

	Unknown Object (File)
	Fri, May 17, 9:47 PM

View All 34 Files

Subscribers

Details

Reviewers

mw
kd
wma
imp
manu

Commits

rG285714ea7a21: libsa/arp.c: Change printf format string
rGc66c6da4f081: libsa/arp.c: Change printf format string

Summary

Change printf format string to avoid compilation termination when ARP_DEBUG macro is defined.

Test Plan

In stand/libsa/arp.c after all #include's and #define's add:
#define ARP_DEBUG
temporary directory to hold compilation files:
mkdir -p $HOME/obj
set appropriate environmental variables, e.g. ARCH=arm64, TARGET_ARCH=aarch64, MAKEOBJDIRPREFIX=$HOME/obj.
In stand/libsa:
sudo -E make buildenv
make
compliation ends with error.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mig_semihalf.com created this revision.Sep 26 2022, 11:02 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 26 2022, 11:02 PM

Herald added a subscriber: dab. · View Herald Transcript

mig_semihalf.com requested review of this revision.Sep 26 2022, 11:02 PM

imp accepted this revision.Sep 26 2022, 11:50 PM

This revision is now accepted and ready to land.Sep 26 2022, 11:50 PM

libsa/arp.c: Change printf format string

Previous revision was created with reversed diff. This
update introduces correct diff.

This revision now requires review to proceed.Sep 28 2022, 1:08 PM

Harbormaster completed remote builds in B47585: Diff 111133.Sep 28 2022, 1:08 PM

ah, that suddenly makes more sense.

This revision is now accepted and ready to land.Sep 29 2022, 2:11 AM

Closed by commit rGc66c6da4f081: libsa/arp.c: Change printf format string (authored by mig_semihalf.com, committed by kd). · Explain WhySep 29 2022, 10:59 AM

This revision was automatically updated to reflect the committed changes.

kd added a commit: rGc66c6da4f081: libsa/arp.c: Change printf format string.

imp added a commit: rG285714ea7a21: libsa/arp.c: Change printf format string.Jan 24 2023, 10:18 PM

Revision Contents
Changeset List

Path

Size

stand/

libsa/

2 lines

Diff 111178

stand/libsa/arp.c

Loading...