Experimental Physics and
Industrial Control System

Noboru Yamamoto <[email protected]> · Fri, 04 Nov 2011 16:21:54 +0900

Hi,

> asyn R4-18 is now available.  Links to the new documentation, release
notes and tar file can be found here:
> http://www.aps.anl.gov/epics/modules/soft/asyn/
I downloaded asyn R4-18 togather with the latest STREAM
device(StreamDevice-2.5).
I can build asynR4-18 on MacOSX Linon10.7.2 with EPICS base 3-14.12.1
without any problem.
However, I got the following warning message to build StreamDevice-2.5:

> ../StreamBusInterface.cc:69:14: warning: variable 'r' is uninitialized when used here [-Wuninitialized]
>     for (r = r->first; r; r = r->next)
>              ^
> ../StreamBusInterface.cc:67:5: note: variable 'r' is declared here
>     StreamBusInterfaceRegistrarBase* r;
>     ^
> ../StreamBusInterface.cc:67:39: note: add initialization to silence this warning
>     StreamBusInterfaceRegistrarBase* r;
>                                       ^
>                                        = 0
> 1 warning generated.
I'm new to the StreamDevice, so I don't know this warning may or may not
cause anyproblem in the real application. But, in general, it seems
better to fix the problem.
Possible solutions are:

1) initialise r with 0, as suggested by compiler:

     StreamBusInterfaceRegistrarBase* r=0;

2) initialise r with StreamBusInterfaceRegistrarBase::first which
defined in StreamBusInterface.cc

     StreamBusInterfaceRegistrarBase* r=
StreamBusInterfaceRegistrarBase::first;

I believe it better to use 2nd option, but I would like to ask gurus in
the collaboration if threre is a reason I should not do this way.

Thanks,

Noboru Yamamoto
Acc. control group
J-PARC center/KEK & JAEA.

(11/11/04 8:44), Mark Rivers wrote:
> Folks,
> 
> asyn R4-18 is now available.  Links to the new documentation, release notes and tar file can be found here:
> http://www.aps.anl.gov/epics/modules/soft/asyn/
> 
> asyn is now also available via the APS Subversion repository here:
> https://svn.aps.anl.gov/epics/asyn/
> 
> This is from the release notes:
> *********************************************
> asynPortDriver
> Cleaned up logic for callbacks on asynUInt32Digital interface. It was not doing callbacks if the   value had not changed but an interrupt had occured for bits in the mask. This would happen when interrupts were only enabled on the falling or rising edges but not both. Added an additional form of setUIntDigitalParam that takes an interruptMask argument. In the previous releases drivers were calling setUInt32Interrupt for callbacks which is not correct. That function should only be called by device support to tell the driver which interrupts to recognize. These problems made the quadEM not work correctly with the ipUnidig.
> 
> drvAsynSerialPort
> Bug fix on WIN32. It was waiting forever when timeout=0 when it should return whatever characters are available without waiting at all.
> 
> Miscellaneous
> Changes to avoid compiler warnings on 64-bit Darwin.
> 
> Changes to avoid compiler warnings and errors on older Solaris compiler.
> 
> Changed non-standard __VAR_ARGS__ to __VA_ARGS__ in asynPrint macro.
> 
> Added titles to EDM screens.
> 
> Added CSS BOY screens.
> 
> Fixed build problem. It was rebuilding the VXI11 code with rpcgen each time make was run, even if nothing had changed.
> 
> ****************************************************
> 
> Mark
> 
> 

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System