Experimental Physics and
Industrial Control System

Andrew Johnson via Core-talk <core-talk at aps.anl.gov> · Thu, 11 Mar 2021 23:04:07 -0000

So gdd is code that we would have thrown away years ago if we didn't
need it to support the CA Gateway and any other CA server tools that use
the PCAS. Those libraries were moved to an external module
https://github.com/epics-modules/pcas before we released EPICS 7.0.1 so
the community still owns and needs to keep the code working, but as to
whether it's worth fixing these I would put on the low end of the
priority order.

About the bi/bo string thing, there's a hole in our APIs because the
get_enum_str() method gets passed a pointer to a buffer without also
telling it how big that buffer is. That's probably why the code is using
the source's size, because the author had no way to know – this dates
all the way back to the first commit of this record code in October
1990. This method probably doesn't get used very much, I actually had a
hard time working out how to get the code to call it, but yet it is a
potential problem which the Core group will need to decide what to do
about. I will file a separate bug about that, so please leave that code
as it is, and thanks for raising the question.

-- 
You received this bug notification because you are a member of EPICS
Core Developers, which is subscribed to EPICS Base.
Matching subscriptions: epics-core-list-subscription
https://bugs.launchpad.net/bugs/1905159

Title:
  Fix warnings from recent compiler versions

Status in EPICS Base:
  Triaged

Bug description:
  Builds of EPICS Base using recent versions of gcc, clang and MSVC
  generate compiler warnings, which we try to minimize if possible. Some
  of these warnings can be prevented with suitable code modifications,
  although the code must still build on the older compilers and other
  architectures that we support.

  Both the 3.15 and 7.0 branches can be examined for warnings; fixes
  made on the 3.15 branch get propagated to the 7.0 branch during
  periodic up-merges, so there is no need to make the same changes to
  both branches. Some code only exists on one branch; for example the
  gdd module was unbundled after 3.15, and there are many new modules in
  7.0.

To manage notifications about this bug go to:
https://bugs.launchpad.net/epics-base/+bug/1905159/+subscriptions

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System