[Arsperl-devel] ARSperl enhancements

[Michiel B. <mic...@gm...>]

Hi All,

I was looking into improving the documentation available for ARSperl
and had the following I would like to discuss.
** There are lots of 2.x and 3.x api calls in the current package and
current documentation, this is confusing and the current version is
only meant to compile against the >= 5.0 API . If people would still
need those calls, they can of course just download ARSperl 1.old if
they want to... My proposal is to remove the old calls from the
package and the documentation.

** The OO stuff that is in the package seems to be broken; my proposal
is to remove that as well.

** Some stuff like ars_GetAlertCount and ars_RegisterForAlerts is not
exported in ARS.pm, although it is in the docs and in ARS.xs. If I add
it and recompile; a sample script that just uses ars_GetAlertCount
gives
"Attempt to free unreferenced scalar: SV 0x187e224, Perl interpreter: 0x223e5c."
It looks like these calls were not completely implemented? My proposal
is to either remove or fix.

** Also; I altered  ars_APIVersion: it now uses AR_CURRENT_API_VERSION
rather than AR_EXPORT_VERSION so the version level that comes back is
more useful.
It now matches the Minimum-Api-Version setting in ar.conf. This seemed
better to me; however I might be off here; especially since I noticed
AR_EXPORT_VERSION being used in ARS.xs all over the place. Please
comment.

Please note that I do not want to delete hard work of people, but I
think that ARSperl is really useful and that it would be nice to have
a clean and all-working version; that would improve the quality of the
product in my opinion.

Kind regards,

Michiel