Re: [certi-dev] Comments regarding hla 1.3 imlpementation in Certi 3.4

[Eric Noulard]

2011/7/20 Jean-Philippe Lebel <address@hidden>:
> Hi Guys,
>
> I received a request to support CERTI 1.4 for our products HLA Toolbox for
> Matlab and HLA Blockset for Simulink.

Good to know.
May be a typo but CERTI version should be 3.4.x and not 1.4.

> First of all, I must say I was surprised by the overall quality of the RTI
> since version 3.2. But, (there is always a but), I had two issues I think
> could be resolved so Certi becomes binary compatible with other HLA 1.3 RTI.
>
> In the Exception class, you added a new function "virtual long getType()=0"
> which is not defined in the original standard.

Right, I think this can be safely removed.
I'll do that soon.

> Also, the Windows build generates RTI-NG.dll and FedTime.dll but I think the
> standard defines them as libRTI-NG.dll and libFedTime.dll

I'll check the DLC document but you may change that by either:

set(CMAKE_SHARED_LIBRARY_PREFIX "lib") in the main CMakeList.txt

and then all generated DLL will be prefixed with "lib".

or do it only for RTI-NG and FedTime within libRTI/CMakeLists.txt

SET_TARGET_PROPERTIES(RTI PROPERTIES OUTPUT_NAME "libRTI-NG")
SET_TARGET_PROPERTIES(FedTime PROPERTIES OUTPUT_NAME "libFedTime")

> Keep on the good work!

Thanks we try our best.

Thank you very much for your report, may be next time you could
file bug report directly on the tracker:
https://savannah.nongnu.org/bugs/?group=certi

that way it's easier for us to track request and feed appropriate
ChangeLog between release.

Moreover, if you open a Savannah account and login before filing the
report you'll get automatic
e-mail follow-up of your request.

If you go a step further and provide a patch proposal with your
request I would be more
than happy to review it.


-- 
Erk
Membre de l'April - « promouvoir et défendre le logiciel libre » -
http://www.april.org