Project

General

Profile

Actions
Copy link

Cleanup #62850

closed

warning: void _denc_finish() defined but not used

Added by Laura Flores over 2 years ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
% Done:

0%

Backport:
squid
Reviewed:
Affected Versions:
Pull request ID:
55872
Tags (freeform):
backport_processed
Merge Commit:
459b79a2fd0e01f74699b2d5f275cdbd913aad76
Fixed In:
v19.3.0-598-g459b79a2fd
Released In:
v20.2.0~3341
Upkeep Timestamp:
2025-11-01T01:34:23+00:00
Tags:
low-hanging-fruit

Description

Seen on the latest main SHA (01ef9e5e91e73422cf11f9b49d06815e4ed75c0d):

In file included from ../src/include/encoding.h:41,
                 from ../src/include/compact_map.h:16,
                 from ../src/include/mempool.h:32,
                 from ../src/os/bluestore/bluestore_types.h:23,
                 from ../src/os/bluestore/bluefs_types.h:8,
                 from ../src/os/bluestore/bluefs_types.cc:5:
../src/include/denc.h:1826:15: warning: ‘void _denc_finish(ceph::buffer::v15_2_0::ptr::const_iterator&, __u8*, __u8*, char**, uint32_t*)’ defined but not used [-Wunused-function]
 1826 |   static void _denc_finish(::ceph::buffer::ptr::const_iterator& p,      \
      |               ^~~~~~~~~~~~
../src/include/denc.h:1826:15: note: in definition of macro ‘DENC_HELPERS’
 1826 |   static void _denc_finish(::ceph::buffer::ptr::const_iterator& p,      \
      |               ^~~~~~~~~~~~
../src/include/denc.h:1816:15: warning: ‘void _denc_start(ceph::buffer::v15_2_0::ptr::const_iterator&, __u8*, __u8*, char**, uint32_t*)’ defined but not used [-Wunused-function]
 1816 |   static void _denc_start(::ceph::buffer::ptr::const_iterator& p,       \
      |               ^~~~~~~~~~~
../src/include/denc.h:1816:15: note: in definition of macro ‘DENC_HELPERS’
 1816 |   static void _denc_start(::ceph::buffer::ptr::const_iterator& p,       \
      |               ^~~~~~~~~~~
../src/include/denc.h:1807:15: warning: ‘void _denc_finish(ceph::buffer::v15_2_0::list::contiguous_appender&, __u8*, __u8*, char**, uint32_t*)’ defined but not used [-Wunused-function]
 1807 |   static void _denc_finish(::ceph::buffer::list::contiguous_appender& p, \
      |               ^~~~~~~~~~~~
../src/include/denc.h:1807:15: note: in definition of macro ‘DENC_HELPERS’
 1807 |   static void _denc_finish(::ceph::buffer::list::contiguous_appender& p, \
      |               ^~~~~~~~~~~~
../src/include/denc.h:1797:15: warning: ‘void _denc_start(ceph::buffer::v15_2_0::list::contiguous_appender&, __u8*, __u8*, char**, uint32_t*)’ defined but not used [-Wunused-function]
 1797 |   static void _denc_start(::ceph::buffer::list::contiguous_appender& p, \
      |               ^~~~~~~~~~~
../src/include/denc.h:1797:15: note: in definition of macro ‘DENC_HELPERS’
 1797 |   static void _denc_start(::ceph::buffer::list::contiguous_appender& p, \
      |               ^~~~~~~~~~~

Reproduce by running:

ninja osd

Not sure if it matters, but this was on Centos Stream 8.

Related issues 1 (0 open — 1 closed)

Copied to bluestore - Backport #67541: squid: warning: void _denc_finish() defined but not usedResolvedActions
Actions
Copy link
 #1

Updated by Jamie Pryde about 2 years ago

I've made an attempt to fix this here https://github.com/ceph/ceph/pull/55428

Actions
Copy link
 #2

Updated by Laura Flores about 2 years ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 55428
Actions
Copy link
 #3

Updated by Dan van der Ster over 1 year ago

  • Pull request ID changed from 55428 to 55872

PR 55428 was abandoned

https://github.com/ceph/ceph/pull/55872 replaced it and was merged

Actions
Copy link
 #4

Updated by Igor Fedotov over 1 year ago

  • Backport set to squid
Actions
Copy link
 #5

Updated by Igor Fedotov over 1 year ago

  • Status changed from Fix Under Review to Pending Backport
Actions
Copy link
 #6

Updated by Upkeep Bot over 1 year ago

  • Copied to Backport #67541: squid: warning: void _denc_finish() defined but not used added
Actions
Copy link
 #7

Updated by Upkeep Bot over 1 year ago

  • Tags (freeform) set to backport_processed
Actions
Copy link
 #8

Updated by Igor Fedotov over 1 year ago

  • Status changed from Pending Backport to Resolved
Actions
Copy link
 #9

Updated by Upkeep Bot 8 months ago

  • Merge Commit set to 459b79a2fd0e01f74699b2d5f275cdbd913aad76
  • Fixed In set to v19.3.0-598-g459b79a2fd0
  • Upkeep Timestamp set to 2025-07-11T22:08:38+00:00
Actions
Copy link
 #10

Updated by Upkeep Bot 8 months ago

  • Fixed In changed from v19.3.0-598-g459b79a2fd0 to v19.3.0-598-g459b79a2fd
  • Upkeep Timestamp changed from 2025-07-11T22:08:38+00:00 to 2025-07-14T23:39:58+00:00
Actions
Copy link
 #11

Updated by Upkeep Bot 5 months ago

  • Released In set to v20.2.0~3341
  • Upkeep Timestamp changed from 2025-07-14T23:39:58+00:00 to 2025-11-01T01:34:23+00:00
Actions
Copy link

Also available in: Atom PDF