Re: svn commit: r20449 - in trunk/subversion/bindings/swig/ruby: svn test

On Thu, Jul 06, 2006 at 07:05:06AM -0700, kou@tigris.org wrote:
> Use ClassName rule instead of CONSTANT_NAME rule for error class name.
>
> --- trunk/subversion/bindings/swig/ruby/svn/error.rb (original)
> +++ trunk/subversion/bindings/swig/ruby/svn/error.rb Thu Jul 6 07:05:05 2006
> @@ -1,4 +1,5 @@
> + class #{error_class_name} < Error
> def initialize(message="", file=nil, line=nil)
> super(#{value}, message, file, line)
> end
> end
> + #{error_const_name} = #{error_class_name}

It might be a good idea to add a comment explaining that this is for
backwards compatibility with pre-1.5 bindings.

> --- trunk/subversion/bindings/swig/ruby/test/test_client.rb (original)
> +++ trunk/subversion/bindings/swig/ruby/test/test_client.rb Thu Jul 6 07:05:05 2006
> @@ -125,14 +125,14 @@
> assert(!File.exist?(dir_path))
> ctx.mkdir(dir_path)
> assert(File.exist?(dir_path))
> - assert_raises(Svn::Error::ENTRY_EXISTS) do
> + assert_raises(Svn::Error::EntryExists) do

I assume that these changes are just to keep up with the new style -
they aren't actually required, are they?

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 6 16:25:43 2006