Hi, Philip,

I did address most of the issues. However, the following question is open:

Von: Philip Martin [[email protected]]
> Markus Schaber <[email protected]> writes:

>> +/* Function to prepare a single test file */
>> +
>> +static svn_error_t *
>> +create_test_file(struct test_file_definition_t* definition, apr_pool_t 
>> *pool, apr_pool_t *scratch_pool)
>> +{
>> +  apr_status_t status;
>> +  apr_file_t *file_h;
>> +  apr_off_t midpos = definition->size / 2;
>> +  svn_error_t *err = NULL;
>> +  int i;
>> +
>> +  if (definition->size < 5)
>> +    SVN_ERR_ASSERT(strlen(definition->data) >= definition->size);
>> +  else
>> +    SVN_ERR_ASSERT(strlen(definition->data) >= 5);
>> +
>> +  status = apr_filepath_merge(&definition->created_path,
>> +                TEST_DIR,
>> +                definition->name,
>> +                0,
>> +                pool);

> Do you need to do that?  Could you simply relative paths as is done
> subversion/tests/libsvn_diff/diff-diff3-test.c.

I used path concatenation to pack all of the files into one temporary
directory, so they can be easily cleaned up, and only one svn:ignore
entry is needed. I did model this after how the temporary 
repositories are handled in svn_test_fs.c.

If you think it's better to clutter the directory with the full set of test 
files,
I'll change the code.

Best regards

Markus Schaber
--

We software Automation.

3S-Smart Software Solutions GmbH
Markus Schaber | Developer
Memminger Str. 151 | 87439 Kempten | Germany | Tel. +49-831-54031-0 | Fax 
+49-831-54031-50

Email: [email protected] | Web: http://www.3s-software.com
CoDeSys internet forum: http://forum.3s-software.com
Download CoDeSys sample projects: 
http://www.3s-software.com/index.shtml?sample_projects

Managing Directors: Dipl.Inf. Dieter Hess, Dipl.Inf. Manfred Werner | Trade 
register: Kempten HRB 6186 | Tax ID No.: DE 167014915

Reply via email to