osync_xmlformat_validate() in format plugins

[Daniel Gollub]

Hi,

i just found that osync_xmlformat_validate() is quite expensive, since it
parses for each call the XSLT schemas from scratch (xmlSchemaParse()).

Callgraph of sync with vformat plugin and the vcard_to_xmlformat converter:
http://cryptomilch.de/~dgollub/OpenSync/opensync_profiling_xmlformat_validate.png
(Look for the redline in on the left bottom of the callgraph - 991x stands for
the 991 function calls.)

Actually we could do this XSLT schema parsing within in the OpenSync framework
once, and validate all xmlformats which got reproted with the same object
from a single xmlSchemaParse() call. This would require some changes for
osync_xmlformat_*() - i try to prepare a patch within the next days. This
would save us 990 of 991 xmlSchemaParse() calls in mentioned example. One
call for each used xmlformat. If someone would sync contacts and events this
would end up in two xmlSchemParse() calls for xmlformat-event and
xmlformat-contact....

Anyway- for now i would recommend to remove osync_xmlformat_validate() calls
from every format plugin converter function. It's O.K. for debugging tools
like "vconvert", but the plugin format conversion routines should be fast ;)

I already removed osync_xmlformat_validate() from the vformat plugin and from
gnokii-format and palm-format plugins. AFAIK there is only opie-sync and
sync4j-sync (is anyone still using this?) left which have
osync_xmlformat_validate(), at least in the OpenSync SVN - didn't checked yet
other format plugins.

best regards,
Daniel

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel

[Daniel Gollub]

On Saturday 03 May 2008 17:16:45 Daniel Gollub wrote:
> i just found that osync_xmlformat_validate() is quite expensive, since it
> parses for each call the XSLT schemas from scratch (xmlSchemaParse()).
Ooops, XML schema - abbrev. XSD .. not XSLT
[...]

> Actually we could do this XSLT schema parsing within in the OpenSync
> framework once, and validate all xmlformats which got reproted with the
Again, XML schema - abbrev. XSD .. not XSLT
[...]

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel

[Bjoern Ricks]

Hi Daniel,

I started coding to get rid of osync_xml_validate and to have only one
instance of a xml schema for each objtype. Therefore I tried to reuse
the oop singleton pattern in C via a static GList. Could you please
review my first intuition because I am more familiar with C++ than C.

best regards,
Bjoern

> Hi,
>
> i just found that osync_xmlformat_validate() is quite expensive, since it
> parses for each call the XSLT schemas from scratch (xmlSchemaParse()).
>
> Callgraph of sync with vformat plugin and the vcard_to_xmlformat converter:
> http://cryptomilch.de/~dgollub/OpenSync/opensync_profiling_xmlformat_validate.png
> (Look for the redline in on the left bottom of the callgraph - 991x stands for
> the 991 function calls.)
>
> Actually we could do this XSLT schema parsing within in the OpenSync framework
> once, and validate all xmlformats which got reproted with the same object
> from a single xmlSchemaParse() call. This would require some changes for
> osync_xmlformat_*() - i try to prepare a patch within the next days. This
> would save us 990 of 991 xmlSchemaParse() calls in mentioned example. One
> call for each used xmlformat. If someone would sync contacts and events this
> would end up in two xmlSchemParse() calls for xmlformat-event and
> xmlformat-contact....
>
> Anyway- for now i would recommend to remove osync_xmlformat_validate() calls
> from every format plugin converter function. It's O.K. for debugging tools
> like "vconvert", but the plugin format conversion routines should be fast ;)
>
> I already removed osync_xmlformat_validate() from the vformat plugin and from
> gnokii-format and palm-format plugins. AFAIK there is only opie-sync and
> sync4j-sync (is anyone still using this?) left which have
> osync_xmlformat_validate(), at least in the OpenSync SVN - didn't checked yet
> other format plugins.
>
> best regards,
> Daniel
>  

--
/Bjoern Ricks

Index: opensync_xmlformat.c
===================================================================
--- opensync_xmlformat.c (Revision 3370)
+++ opensync_xmlformat.c (Arbeitskopie)
@@ -91,6 +91,80 @@
  return p;
 }
 
+OSyncXMLFormatSchema * osync_xmlformat_schema_get_instance(OSyncXMLFormat * xmlformat, OSyncError **error) {
+ static GList * schemas = NULL;
+ OSyncXMLFormatSchema * osyncschema = NULL;
+ GList * entry;
+ const char * objtype;
+
+ osync_trace(TRACE_ENTRY, "%s(%p, %p)", __func__, xmlformat, error);
+
+ osync_assert(xmlformat);
+
+ objtype = osync_xmlformat_get_objtype(xmlformat);
+ //TODO get mutex
+ // find schema for objtype
+ for ( entry = schemas; entry != NULL; entry=entry->next) { // should be fast enough for only a few objtypes
+ osyncschema = (OSyncXMLFormatSchema *) entry->data;
+ if (!strcmp(osyncschema->objtype, objtype) ) {
+ return osyncschema;
+ }
+ osyncschema = NULL;
+ }
+ if ( osyncschema == NULL ) {
+ osyncschema = osync_try_malloc0(sizeof(OSyncXMLFormatSchema), error);
+ if(!osyncschema) {
+ osync_trace(TRACE_EXIT_ERROR, "%s: %s" , __func__, osync_error_print(error));
+ //TODO release mutex
+ return NULL;
+ }
+ osyncschema->objtype = g_strdup(objtype);
+ char *schemafilepath = g_strdup_printf("%s%c%s%s%s",
+ OPENSYNC_SCHEMASDIR,
+ G_DIR_SEPARATOR,
+ "xmlformat-",
+ osyncschema->objtype,
+ ".xsd");
+
+ xmlSchemaParserCtxtPtr xmlSchemaParserCtxt;
+ xmlSchemaPtr xmlSchema;
+ xmlSchemaValidCtxtPtr xmlSchemaValidCtxt;
+
+ xmlSchemaParserCtxt = xmlSchemaNewParserCtxt(schemafilepath);
+ osyncschema->schema = xmlSchemaParse(xmlSchemaParserCtxt);
+ xmlSchemaFreeParserCtxt(xmlSchemaParserCtxt);
+
+ osyncschema->context = xmlSchemaNewValidCtxt(xmlSchema);
+ if (osyncschema->context == NULL) {
+ xmlSchemaFree(osyncschema->schema);
+ g_free(osyncschema->objtype);
+ g_free(osyncschema);
+ osyncschema = NULL;
+ //TODO set error
+ }
+ schemas = g_list_append(schemas, osyncschema);
+ }
+ //TODO release mutex
+ return osyncschema;
+
+}
+
+
+osync_bool osync_xmlformat_schema_validate(OSyncXMLFormatSchema * schema, OSyncXMLFormat * xmlformat)
+{
+ osync_assert(xmlformat);
+ osync_assert(schema);
+
+ int rc = 0;
+
+ /* Validate the document */
+ rc = xmlSchemaValidateDoc(schema->context, xmlformat->doc);
+
+ if(rc != 0)
+ return FALSE;
+ return TRUE;
+}
+
 /**
  * @brief Validate the xmlformat against its schema in inidivual path
  * @param xmlformat The pointer to a xmlformat object
@@ -399,8 +473,9 @@
 osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat)
 {
  osync_assert(xmlformat);
-
- return _osync_xmlformat_validate(xmlformat, NULL);
+
+ OSyncXMLFormatSchema * schema = osync_xmlformat_schema_get_instance(xmlformat, NULL);
+ return osync_xmlformat_schema_validate(schema, xmlformat);
 }
 
 /**
Index: opensync_xmlformat_internals.h
===================================================================
--- opensync_xmlformat_internals.h (Revision 3370)
+++ opensync_xmlformat_internals.h (Arbeitskopie)
@@ -45,7 +45,20 @@
 
 };
 
+struct OSyncXMLFormatSchema {
+
+ xmlSchemaPtr schema;
+
+ xmlSchemaValidCtxtPtr context;
+
+ const char *objtype
+};
+
 int _osync_xmlformat_get_points(OSyncXMLPoints points[], int* cur_pos, int basic_points, const char* fieldname);
 osync_bool _osync_xmlformat_validate(OSyncXMLFormat *xmlformat, const char *path);
 
+OSyncXMLFormatSchema * osync_xmlformat_schema_get_instance(OSyncXMLFormat * xmlformat, OSyncError **error)
+void osync_xmlformat_schema_free_instance(OSyncXMLFormatSchema *  schema, OSyncError **error);
+osync_bool osync_xmlformat_schema_validate(OSyncXMLFormatSchema * schema, OSyncXMLFormat * xmlformat);
+
 #endif /*OPENSYNC_XMLFORMAT_INTERNAL_H_*/

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel

[Daniel Gollub]

On Thursday 19 June 2008 16:18:02 Bjoern Ricks wrote:
> I started coding to get rid of osync_xml_validate and to have only one
> instance of a xml schema for each objtype. Therefore I tried to reuse
> the oop singleton pattern in C via a static GList. Could you please
> review my first intuition because I am more familiar with C++ than C.

Thanks for looking into this!

The idea of using C++-singleton-like sounds interesting. I wonder if we would
change some more stuff, if this would be still required. For example make all
OSyncXMLFormat "inherit" of OSyncXMLFormatSchema? We just force/require for
all XMLFormats a schema file, by changing osync_xmlformat_new parameter list
to:

osync_xmlformat_new(const char *objtype, OSyncXMLFormatSchema *schema,
OSyncError **error);

And putting an OSyncXMLFormatSchema pointer into the internal struct.

Thoughts?

[...]

>  /**
>   * @brief Validate the xmlformat against its schema in inidivual path
>   * @param xmlformat The pointer to a xmlformat object
> @@ -399,8 +473,9 @@
>  osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat)
>  {
>         osync_assert(xmlformat);
> -
> -       return _osync_xmlformat_validate(xmlformat, NULL);
> +
> +       OSyncXMLFormatSchema * schema =
> osync_xmlformat_schema_get_instance(xmlformat, NULL); +       return
> osync_xmlformat_schema_validate(schema, xmlformat);

We should avoid code which is ignoring the error handling by passing NULL
instead of a OSyncError reference. (I know there is/was some code doing
so ... but we should get rid of those old code. It's just bad/wrong/....).
Feel free to change the paramter list vor the _validate() function to
(OSyncXMLFormat *xmlformat, OSyncError **error). You might set also an error
when osync_xmlformat_schema_validate is false:
osync_error_set(*error, OSYNC_ERROR_GENERIC /*or something different */, "XML
Format validation failed.");

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel

[Bjoern Ricks]

> On Thursday 19 June 2008 16:18:02 Bjoern Ricks wrote:
>  
>> I started coding to get rid of osync_xml_validate and to have only one
>> instance of a xml schema for each objtype. Therefore I tried to reuse
>> the oop singleton pattern in C via a static GList. Could you please
>> review my first intuition because I am more familiar with C++ than C.
>>    
>
> Thanks for looking into this!
>
> The idea of using C++-singleton-like sounds interesting. I wonder if we would
> change some more stuff, if this would be still required. For example make all
> OSyncXMLFormat "inherit" of OSyncXMLFormatSchema? We just force/require for
> all XMLFormats a schema file, by changing osync_xmlformat_new parameter list
> to:
>
> osync_xmlformat_new(const char *objtype, OSyncXMLFormatSchema *schema,
> OSyncError **error);
>
> And putting an OSyncXMLFormatSchema pointer into the internal struct.
>
> Thoughts?
>
> [...]
>  

That could be a better solution than to use static variables for each
objtype. I was thinking about that too. But I didn't dare to change the
parameter list.
But the new solution has another problem. Where should the
OSyncXMLFormatSchemas be created (and released)? We need only one
instance per objtype. Is there one osnyc engine per application? Then
the engine could be the right part to create OSyncXMLFormatSchemas.

>
>  
>>  /**
>>   * @brief Validate the xmlformat against its schema in inidivual path
>>   * @param xmlformat The pointer to a xmlformat object
>> @@ -399,8 +473,9 @@
>>  osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat)
>>  {
>>         osync_assert(xmlformat);
>> -
>> -       return _osync_xmlformat_validate(xmlformat, NULL);
>> +
>> +       OSyncXMLFormatSchema * schema =
>> osync_xmlformat_schema_get_instance(xmlformat, NULL); +       return
>> osync_xmlformat_schema_validate(schema, xmlformat);
>>    
>
> We should avoid code which is ignoring the error handling by passing NULL
> instead of a OSyncError reference. (I know there is/was some code doing
> so ... but we should get rid of those old code. It's just bad/wrong/....).
> Feel free to change the paramter list vor the _validate() function to
> (OSyncXMLFormat *xmlformat, OSyncError **error). You might set also an error
> when osync_xmlformat_schema_validate is false:
> osync_error_set(*error, OSYNC_ERROR_GENERIC /*or something different */, "XML
> Format validation failed.");
>  

:-) I was irritated because osync_xmlformat_validate had no error
parameter. Therefore I had to use a NULL value. So we should change the
function signature.
--
------------------------------------------------------------------------
University of Applied Sciences Osnabrueck
Faculty of Engineering and Computer Science
Dipl.-Inf. (FH) Bjoern Ricks

P.O. Box 1940 - 49009 Osnabrueck - Germany
Phone: [phone] Fax: [phone]
Email: b.ricks@... <mailto:b.ricks@...>
Web: http://www.ecs.fh-osnabrueck.de/ricks.html

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel

[Bjoern Ricks]

Hi Daniel,

> On Thursday 19 June 2008 16:18:02 Bjoern Ricks wrote:
>  
>> I started coding to get rid of osync_xml_validate and to have only one
>> instance of a xml schema for each objtype. Therefore I tried to reuse
>> the oop singleton pattern in C via a static GList. Could you please
>> review my first intuition because I am more familiar with C++ than C.
>>    
>
> Thanks for looking into this!
>  

You are welcome ;-)

> The idea of using C++-singleton-like sounds interesting. I wonder if we would
> change some more stuff, if this would be still required. For example make all
> OSyncXMLFormat "inherit" of OSyncXMLFormatSchema? We just force/require for
> all XMLFormats a schema file, by changing osync_xmlformat_new parameter list
> to:
>
> osync_xmlformat_new(const char *objtype, OSyncXMLFormatSchema *schema,
> OSyncError **error);
>
> And putting an OSyncXMLFormatSchema pointer into the internal struct.
>
> Thoughts?
>
>
>  

The last days I was thinking about that and it is alway better not to

use any static variables because of concurrent access. But I don't think
we could avoid this problem without having more schema instances in the
memory. If we use your solution every format plugin has to create its
own instance of each schema which the plugin will support. This will be
better than today but not really nice.

Maybe you can review my code again and test it. I changed the xmlformat
test suite to fit with the new solution, too. If my code is ok I will
commit it. Afterwards I take a look at the format plugins and change
them if necessary.

>>  /**
>>   * @brief Validate the xmlformat against its schema in inidivual path
>>   * @param xmlformat The pointer to a xmlformat object
>> @@ -399,8 +473,9 @@
>>  osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat)
>>  {
>>         osync_assert(xmlformat);
>> -
>> -       return _osync_xmlformat_validate(xmlformat, NULL);
>> +
>> +       OSyncXMLFormatSchema * schema =
>> osync_xmlformat_schema_get_instance(xmlformat, NULL); +       return
>> osync_xmlformat_schema_validate(schema, xmlformat);
>>    
>
> We should avoid code which is ignoring the error handling by passing NULL
> instead of a OSyncError reference. (I know there is/was some code doing
> so ... but we should get rid of those old code. It's just bad/wrong/....).
> Feel free to change the paramter list vor the _validate() function to
> (OSyncXMLFormat *xmlformat, OSyncError **error). You might set also an error
> when osync_xmlformat_schema_validate is false:
> osync_error_set(*error, OSYNC_ERROR_GENERIC /*or something different */, "XML
> Format validation failed.");
>  

changed.

--
/Bjoern Ricks

Index: opensync/xmlformat/opensync_xmlformat.c
===================================================================
--- opensync/xmlformat/opensync_xmlformat.c (Revision 3381)
+++ opensync/xmlformat/opensync_xmlformat.c (Arbeitskopie)
@@ -97,6 +97,7 @@
  * @param path The individual schema path. If NULL the default OPENSYNC_SCHEMASDIR is used.
  * @return TRUE if xmlformat valid else FALSE
  */
+/*
 osync_bool _osync_xmlformat_validate(OSyncXMLFormat *xmlformat, const char *path)
 {
  osync_assert(xmlformat);
@@ -111,8 +112,141 @@
  g_free(schemafilepath);
 
  return res;
+} */
+
+/**
+ * @brief Create new OSyncXMLFormatSchema for xmlformat
+ * @param xmlformat The pointer to a xmlformat object. xmlformat->objtype is used to identify the schema file
+ * @param path The individual schema path. If NULL the default OPENSYNC_SCHEMASDIR is used.
+ * @param error The error which will hold the info in case of an error
+ * @return new OSyncXMLFormatSchema or NULL in case of an error
+ */
+OSyncXMLFormatSchema * osync_xmlformat_schema_new(OSyncXMLFormat * xmlformat, const char *path, OSyncError **error) {
+ OSyncXMLFormatSchema * osyncschema = NULL;
+
+ osync_trace(TRACE_ENTRY, "%s(%p, %p, %p)", __func__, xmlformat, path, error);
+
+ osync_assert(xmlformat);
+
+ osyncschema = osync_try_malloc0(sizeof(OSyncXMLFormatSchema), error);
+ if(!osyncschema) {
+ osync_trace(TRACE_EXIT_ERROR, "%s: %s" , __func__, osync_error_print(error));
+ // release mutex
+ return NULL;
+ }
+ osyncschema->objtype = g_strdup(osync_xmlformat_get_objtype(xmlformat));
+ char *schemafilepath = g_strdup_printf("%s%c%s%s%s",
+ path ? path : OPENSYNC_SCHEMASDIR,
+ G_DIR_SEPARATOR,
+ "xmlformat-",
+ osyncschema->objtype,
+ ".xsd");
+
+ xmlSchemaParserCtxtPtr xmlSchemaParserCtxt;
+ xmlSchemaPtr xmlSchema;
+ xmlSchemaValidCtxtPtr xmlSchemaValidCtxt;
+
+ xmlSchemaParserCtxt = xmlSchemaNewParserCtxt(schemafilepath);
+ osyncschema->schema = xmlSchemaParse(xmlSchemaParserCtxt);
+ xmlSchemaFreeParserCtxt(xmlSchemaParserCtxt);
+
+ osyncschema->context = xmlSchemaNewValidCtxt(osyncschema->schema);
+ if (osyncschema->context == NULL) {
+ xmlSchemaFree(osyncschema->schema);
+ g_free(osyncschema->objtype);
+ g_free(osyncschema);
+ osyncschema = NULL;
+ osync_error_set(error, OSYNC_ERROR_GENERIC, "XMLFormat validation failed. Could not create Schema Context.");
+ }
+ return osyncschema;
 }
 
+/**
+ * @brief Free existing OSyncXMLFormatSchema
+ * @param osyncschema Pointer to the Schema that shoud be freed
+ * @param error The error which will hold the info in case of an error
+ */
+void osync_xmlformat_schema_free(OSyncXMLFormatSchema * osyncschema) {
+
+ osync_assert(osyncschema);
+
+ xmlSchemaFreeValidCtxt(osyncschema->context);
+ xmlSchemaFree(osyncschema->schema);
+ g_free(osyncschema->objtype);
+ g_free(osyncschema);
+
+}
+
+/**
+ * @brief Get a  schema for the xmlformat.
+ *
+ * This function creates only one instance of a schema for each objtype. If a xmlformat is passed as a parameter with the same
+ * objtype as a xmlformat prior the returned pointer to a OSyncXMLFormatSchema instance is the same as before.
+ *
+ * @param xmlformat The pointer to a xmlformat object
+ * @param error The error which will hold the info in case of an error
+ * @return Pointer to a instance of OSyncXMLFormatSchema
+ */
+
+OSyncXMLFormatSchema * osync_xmlformat_schema_get_instance(OSyncXMLFormat * xmlformat, OSyncError **error) {
+ static GList * schemas = NULL;
+ static GStaticMutex mutex = G_STATIC_MUTEX_INIT;
+ OSyncXMLFormatSchema * osyncschema = NULL;
+ GList * entry;
+ const char * objtype;
+
+ osync_trace(TRACE_ENTRY, "%s(%p, %p)", __func__, xmlformat, error);
+
+ osync_assert(xmlformat);
+
+ objtype = osync_xmlformat_get_objtype(xmlformat);
+ // get mutex
+ g_static_mutex_lock(&mutex);
+ // find schema for objtype
+ for ( entry = schemas; entry != NULL; entry=entry->next) { // should be fast enough for only a few objtypes
+ osyncschema = (OSyncXMLFormatSchema *) entry->data;
+ if (!strcmp(osyncschema->objtype, objtype) ) {
+ return osyncschema;
+ }
+ osyncschema = NULL;
+ }
+ if ( osyncschema == NULL ) {
+ osyncschema = osync_xmlformat_schema_new(xmlformat, NULL, error);
+ if ( osyncschema != NULL ) {
+ schemas = g_list_append(schemas, osyncschema);
+ }
+ }
+ // release mutex
+ g_static_mutex_unlock(&mutex);
+ return osyncschema;
+}
+
+/**
+ * @brief Validate the xmlformat against its schema
+ * @param xmlformat The pointer to a xmlformat object
+ * @param schema The pointer to a OSyncXMLFormatSchema object.
+ * @param error The error which will hold the info in case of an error
+ * @return TRUE if xmlformat valid else FALSE
+ */
+
+osync_bool osync_xmlformat_schema_validate(OSyncXMLFormatSchema * schema, OSyncXMLFormat * xmlformat, OSyncError **error)
+{
+ osync_assert(xmlformat);
+ osync_assert(schema);
+
+ int rc = 0;
+
+ /* Validate the document */
+ rc = xmlSchemaValidateDoc(schema->context, xmlformat->doc);
+
+ if(rc != 0) {
+ osync_error_set(error, OSYNC_ERROR_GENERIC, "XMLFormat validation failed.");
+ return FALSE;
+ }
+ return TRUE;
+}
+
+
 /*@}*/
 
 /**
@@ -394,13 +528,15 @@
 /**
  * @brief Validate the xmlformat against its schema
  * @param xmlformat The pointer to a xmlformat object
+ * @param error The error which will hold the info in case of an error
  * @return TRUE if xmlformat valid else FALSE
  */
-osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat)
+osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat, OSyncError **error)
 {
  osync_assert(xmlformat);
-
- return _osync_xmlformat_validate(xmlformat, NULL);
+
+ OSyncXMLFormatSchema * schema = osync_xmlformat_schema_get_instance(xmlformat, error);
+ return osync_xmlformat_schema_validate(schema, xmlformat, error);
 }
 
 /**
Index: opensync/xmlformat/opensync_xmlformat_internals.h
===================================================================
--- opensync/xmlformat/opensync_xmlformat_internals.h (Revision 3381)
+++ opensync/xmlformat/opensync_xmlformat_internals.h (Arbeitskopie)
@@ -23,7 +23,9 @@
 #ifndef OPENSYNC_XMLFORMAT_INTERNALS_H_
 #define OPENSYNC_XMLFORMAT_INTERNALS_H_
 
-#include <opensync/opensync_xml.h>
+#include <libxml/xpath.h>
+#include <libxml/xmlschemas.h>
+#include <libxml/tree.h>
 
 /**
  * @brief Represent a XMLFormat object
@@ -45,7 +47,24 @@
 
 };
 
+/**
+ * @brief Represents a Schema object
+ * @ingroup OSyncXMLFormatPrivateAPI
+ */
+typedef struct OSyncXMLFormatSchema {
+ /** The schema object */
+ xmlSchemaPtr schema;
+ /** The schema validation context */
+ xmlSchemaValidCtxtPtr context;
+ /** The object type of OSyncXMLFormat */
+ char * objtype;
+} OSyncXMLFormatSchema;
+
 int _osync_xmlformat_get_points(OSyncXMLPoints points[], int* cur_pos, int basic_points, const char* fieldname);
-osync_bool _osync_xmlformat_validate(OSyncXMLFormat *xmlformat, const char *path);
 
+OSyncXMLFormatSchema * osync_xmlformat_schema_get_instance(OSyncXMLFormat * xmlformat, OSyncError **error);
+OSyncXMLFormatSchema * osync_xmlformat_schema_new(OSyncXMLFormat * xmlformat, const char *path, OSyncError **error);
+void osync_xmlformat_schema_free(OSyncXMLFormatSchema * schema);
+osync_bool osync_xmlformat_schema_validate(OSyncXMLFormatSchema * schema, OSyncXMLFormat * xmlformat, OSyncError **error);
+
 #endif /*OPENSYNC_XMLFORMAT_INTERNAL_H_*/
Index: opensync/xmlformat/opensync_xmlformat.h
===================================================================
--- opensync/xmlformat/opensync_xmlformat.h (Revision 3381)
+++ opensync/xmlformat/opensync_xmlformat.h (Arbeitskopie)
@@ -48,7 +48,7 @@
 OSYNC_EXPORT OSyncXMLFieldList *osync_xmlformat_search_field(OSyncXMLFormat *xmlformat, const char *name, OSyncError **error, ...);
 
 OSYNC_EXPORT osync_bool osync_xmlformat_assemble(OSyncXMLFormat *xmlformat, char **buffer, unsigned int *size);
-OSYNC_EXPORT osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat);
+OSYNC_EXPORT osync_bool osync_xmlformat_validate(OSyncXMLFormat *xmlformat, OSyncError **error);
 
 OSYNC_EXPORT void osync_xmlformat_sort(OSyncXMLFormat *xmlformat);
 
Index: tests/merger-tests/check_xmlformat.c
===================================================================
--- tests/merger-tests/check_xmlformat.c (Revision 3381)
+++ tests/merger-tests/check_xmlformat.c (Arbeitskopie)
@@ -275,6 +275,7 @@
  char *buffer;
  unsigned int size;
  OSyncError *error = NULL;
+ OSyncXMLFormatSchema *schema = NULL;
 
  fail_unless(osync_file_read("event.xml", &buffer, &size, &error), NULL);
  fail_unless(error == NULL, NULL);
@@ -283,9 +284,13 @@
  fail_unless(error == NULL, NULL);
 
  g_free(buffer);
-
- fail_unless(_osync_xmlformat_validate(xmlformat, testbed) != FALSE, NULL);
-
+ schema = osync_xmlformat_schema_new(xmlformat, testbed, &error);
+ fail_if( schema == NULL );
+ //fail_unless(_osync_xmlformat_validate(xmlformat, testbed) != FALSE, NULL);
+ fail_unless( osync_xmlformat_schema_validate(schema, xmlformat, &error) );
+
+ osync_xmlformat_schema_free(schema);
+
  osync_xmlformat_unref(xmlformat);
 
  destroy_testbed(testbed);

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Opensync-devel mailing list
Opensync-devel@...
https://lists.sourceforge.net/lists/listinfo/opensync-devel