fix(qudarap): prevent call to undefined by guarding with a macro

[plexoos]

See qudarap/QEvt.hh

struct QUDARAP_API QEvt : public SCompProvider
{
...
#ifndef PRODUCTION
...
    NP*      gatherGenstepFromDevice() const ;
...
#endif
...
};

[Copilot]

Pull request overview

This PR prevents calls to undefined functions in production builds by guarding the call to gatherGenstepFromDevice() with the #ifndef PRODUCTION macro. The method gatherGenstepFromDevice() is a debug-only function that downloads genstep data from device memory, and its declaration in the header file is already conditionally compiled out in production builds.

Key Changes:

• Added #ifndef PRODUCTION / #endif guards around the SCOMP_GENSTEP case in the switch statement to prevent calling an undefined function in production builds

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

When PRODUCTION is defined, the SCOMP_GENSTEP case will return nullptr instead of providing the host-side genstep data. Consider uncommenting line 1362 to use getGenstep() as a fallback, or add an #else clause:

#ifndef PRODUCTION
        case SCOMP_GENSTEP:       a = gatherGenstepFromDevice() ; break ;
#else
        case SCOMP_GENSTEP:       a = getGenstep() ; break ;
#endif

This would ensure that genstep data remains accessible in production builds via the host-side copy, while avoiding the debug-only device download.

Suggested change

	//case SCOMP_GENSTEP: a = getGenstep() ; break ;
	#ifndef PRODUCTION
	case SCOMP_GENSTEP: a = gatherGenstepFromDevice() ; break ;
	#ifndef PRODUCTION
	case SCOMP_GENSTEP: a = gatherGenstepFromDevice() ; break ;
	#else
	case SCOMP_GENSTEP: a = getGenstep() ; break ;