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

	Unknown Object (File)
	Wed, Dec 10, 6:16 AM

	Unknown Object (File)
	Sun, Dec 7, 1:57 PM

	Unknown Object (File)
	Sat, Dec 6, 5:34 AM

	Unknown Object (File)
	Sat, Dec 6, 4:11 AM

	Unknown Object (File)
	Mon, Dec 1, 4:58 AM

	Unknown Object (File)
	Tue, Nov 25, 10:29 PM

	Unknown Object (File)
	Tue, Nov 25, 2:27 AM

	Unknown Object (File)
	Nov 8 2025, 1:32 AM

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

Lint

Lint Skipped

Unit

Tests Skipped

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 111032

stand/libsa/arp.c

Loading...