r2corba 1.5.4 → 1.6.1

data/ext/librpoa/servant.cpp

@@ -11,7 +11,6 @@
 #------------------------------------------------------------------*/
 
 #include "poa.h"
-#include "ace/Auto_Ptr.h"
 #include "tao/DynamicInterface/Server_Request.h"
 #include "tao/DynamicInterface/Dynamic_Implementation.h"
 #include "tao/AnyTypeCode/Any.h"
@@ -25,10 +24,8 @@
 #include "exception.h"
 #include "orb.h"
 #include "servant.h"
-
-#if RPOA_NEED_DSI_FIX
-# include "srvreq_fix.cpp"
-#endif
+#include <memory>
+#include <cstring>
 
 #define RUBY_INVOKE_FUNC RUBY_ALLOC_FUNC
 
@@ -74,7 +71,7 @@ void r2tao_init_Servant()
   VALUE klass;
 
   r2tao_cServant = klass = rb_eval_string("::R2CORBA::PortableServer::Servant");
-  rb_define_alloc_func (r2tao_cServant, RUBY_ALLOC_FUNC (srv_alloc));
+  rb_define_alloc_func (r2tao_cServant, srv_alloc);
   rb_define_method(klass, "_default_POA", RUBY_METHOD_FUNC(r2tao_Servant_default_POA), 0);
   rb_define_method(klass, "_this", RUBY_METHOD_FUNC(r2tao_Servant_this), 0);
 
@@ -98,6 +95,7 @@ void r2tao_init_Servant()
   interface_repository_id_ID = rb_intern ("_interface_repository_id");
 
   r2tao_cDynamicImp = klass = rb_eval_string("::R2CORBA::PortableServer::DynamicImplementation");
+  rb_define_alloc_func (r2tao_cDynamicImp, srv_alloc);
 
   invoke_ID = rb_intern ("invoke");
   primary_interface_ID = rb_intern ("_primary_interface");
@@ -113,15 +111,15 @@ void r2tao_init_Servant()
 //===================================================================
 
 struct DSI_Data {
-  R2CORBA_ServerRequest_ptr  _request;
+  CORBA::ServerRequest_ptr  _request;
   CORBA::NVList_ptr _nvlist;
   CORBA::TypeCode_var _result_type;
   VALUE _rData;
 
-  DSI_Data(R2CORBA_ServerRequest_ptr _req)
-    : _request(_req), _nvlist(0), _rData(Qnil) {}
+  DSI_Data(CORBA::ServerRequest_ptr _req)
+    : _request(_req), _nvlist(nullptr), _rData(Qnil) {}
   ~DSI_Data() {
-    if (this->_rData!=Qnil) { DATA_PTR(this->_rData) = 0; }
+    if (this->_rData!=Qnil) { DATA_PTR(this->_rData) = nullptr; }
   }
 };
 
@@ -134,7 +132,7 @@ VALUE r2tao_ServerRequest_operation(VALUE self)
 {
   if (DATA_PTR (self) != 0)
   {
-    R2CORBA_ServerRequest_ptr request = static_cast<DSI_Data*> (DATA_PTR (self))->_request;
+    CORBA::ServerRequest_ptr request = static_cast<DSI_Data*> (DATA_PTR (self))->_request;
     return rb_str_new2 (request->operation ());
   }
   return Qnil;
@@ -152,7 +150,7 @@ VALUE r2tao_ServerRequest_describe(VALUE self, VALUE desc)
       X_CORBA (BAD_INV_ORDER);
     }
 
-    R2CORBA_ServerRequest_ptr request = dsi_data->_request;
+    CORBA::ServerRequest_ptr request = dsi_data->_request;
 
     if (desc != Qnil && rb_type (desc) == T_HASH)
     {
@@ -289,11 +287,11 @@ VALUE r2tao_ServerRequest_get(VALUE self, VALUE key)
     if (rb_obj_is_kind_of (key, rb_cString) == Qtrue)
     {
       char* arg_name = RSTRING_PTR (key);
-      CORBA::ULong arg_num = dsi_data->_nvlist->count ();
-      for (CORBA::ULong ix=0; ix<arg_num ;++ix)
+      CORBA::ULong const arg_num = dsi_data->_nvlist->count ();
+      for (CORBA::ULong ix=0; ix < arg_num ;++ix)
       {
         CORBA::NamedValue_ptr _nv = dsi_data->_nvlist->item (ix);
-        if (_nv->name () && ACE_OS::strcmp (arg_name, _nv->name ()) == 0)
+        if (_nv->name () && std::strcmp (arg_name, _nv->name ()) == 0)
         {
           R2TAO_TRY
           {
@@ -343,11 +341,11 @@ VALUE r2tao_ServerRequest_set(VALUE self, VALUE key, VALUE val)
     if (rb_obj_is_kind_of (key, rb_cString) == Qtrue)
     {
       char* arg_name = RSTRING_PTR (key);
-      CORBA::ULong arg_num = dsi_data->_nvlist->count ();
-      for (CORBA::ULong ix=0; ix<arg_num ;++ix)
+      CORBA::ULong const arg_num = dsi_data->_nvlist->count ();
+      for (CORBA::ULong ix = 0; ix < arg_num ;++ix)
       {
         CORBA::NamedValue_ptr _nv = dsi_data->_nvlist->item (ix);
-        if (_nv->name () && ACE_OS::strcmp (arg_name, _nv->name ()) == 0)
+        if (_nv->name () && std::strcmp (arg_name, _nv->name ()) == 0)
         {
           R2TAO_TRY
           {
@@ -483,87 +481,20 @@ VALUE DSI_Servant::_invoke_implementation(VALUE args)
 
 DSI_Servant::METHOD  DSI_Servant::method_id (const char* method)
 {
-  if (ACE_OS::strcmp (method, "_is_a") == 0)
+  if (std::strcmp (method, "_is_a") == 0)
     return IS_A;
-  else if (ACE_OS::strcmp (method, "_repository_id") == 0)
+  else if (std::strcmp (method, "_repository_id") == 0)
     return REPOSITORY_ID;
-  else if (ACE_OS::strcmp (method, "_non_existent") == 0)
+  else if (std::strcmp (method, "_non_existent") == 0)
     return NON_EXISTENT;
-  else if (ACE_OS::strcmp (method, "_component") == 0)
+  else if (std::strcmp (method, "_component") == 0)
     return GET_COMPONENT;
-  else if (ACE_OS::strcmp (method, "_interface") == 0)
+  else if (std::strcmp (method, "_interface") == 0)
     return GET_INTERFACE;
 
   return NONE;
 }
 
-#if RPOA_NEED_DSI_FIX
-void DSI_Servant::invoke (CORBA::ServerRequest_ptr /*request*/)
-{}
-
-void DSI_Servant::_dispatch (TAO_ServerRequest &request,
-                             void * /*context*/)
-{
-  // No need to do any of this if the client isn't waiting.
-  if (request.response_expected ())
-    {
-      if (request.is_forwarded ())
-        {
-          request.init_reply ();
-          request.tao_send_reply ();
-
-          // No need to invoke in this case.
-          return;
-        }
-      else if (request.sync_with_server ())
-        {
-          // The last line before the call to this function
-          // was an ACE_CHECK_RETURN, so if we're here, we
-          // know there is no exception so far, and that's all
-          // a SYNC_WITH_SERVER client request cares about.
-          request.send_no_exception_reply ();
-        }
-    }
-
-  // Create DSI request object.
-  R2CORBA::ServerRequest *dsi_request = 0;
-  ACE_NEW (dsi_request,
-           R2CORBA::ServerRequest (request));
-
-  try
-    {
-      // Delegate to user.
-      this->invoke_fix (dsi_request);
-
-      // Only if the client is waiting.
-      if (request.response_expected () && !request.sync_with_server ())
-        {
-          dsi_request->dsi_marshal ();
-        }
-    }
-  catch (::CORBA::Exception& ex)
-    {
-      // Only if the client is waiting.
-      if (request.response_expected () && !request.sync_with_server ())
-        {
-          if (request.collocated ()
-               && request.operation_details ()->cac () != 0)
-            {
-              // If we have a cac it will handle the exception and no
-              // need to do any further processing
-              request.operation_details ()->cac ()->handle_corba_exception (
-                request, &ex);
-              return;
-            }
-          else
-            request.tao_send_reply_exception (ex);
-        }
-    }
-
-  ::CORBA::release (dsi_request);
-}
-#endif
-
 // invocation helper for threadsafe calling of Ruby code
 void* DSI_Servant::thread_safe_invoke (void * arg)
 {
@@ -581,24 +512,20 @@ void* DSI_Servant::thread_safe_invoke (void * arg)
   return 0;
 }
 
-# if RPOA_NEED_DSI_FIX
-void DSI_Servant::invoke_fix (R2CORBA::ServerRequest_ptr request)
-# else
 void DSI_Servant::invoke (CORBA::ServerRequest_ptr request)
-# endif
 {
   ThreadSafeArg tca_(this, request);
 
   void* rc = r2tao_call_thread_safe (DSI_Servant::thread_safe_invoke, &tca_);
-  if (rc != 0)
+  if (rc != nullptr)
   {
     CORBA::SystemException* exc = reinterpret_cast<CORBA::SystemException*> (rc);
-    ACE_Auto_Basic_Ptr<CORBA::SystemException> e_ptr(exc);
+    std::unique_ptr<CORBA::SystemException> e_ptr(exc);
     exc->_raise ();
   }
 }
 
-void DSI_Servant::inner_invoke (R2CORBA_ServerRequest_ptr request)
+void DSI_Servant::inner_invoke (CORBA::ServerRequest_ptr request)
 {
   if (TAO_debug_level > 7)
     ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) - Servant::invoke(%C)\n", request->operation ()));
@@ -628,13 +555,13 @@ void DSI_Servant::inner_invoke (R2CORBA_ServerRequest_ptr request)
       // set ORB arguments (retrieves data for IN/INOUT args)
       request->arguments (nvlist);
 
-      const char *tmp = 0;
+      const char *tmp = nullptr;
       (*_nv->value ()) >>= tmp;
 
       f = this->_is_a (tmp);
 
       if (TAO_debug_level > 5)
-        ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) _is_a (%s) -> %d\n", tmp, f));
+        ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) _is_a (%C) -> %d\n", tmp, f));
     }
     else
     {
@@ -663,7 +590,7 @@ void DSI_Servant::inner_invoke (R2CORBA_ServerRequest_ptr request)
     CORBA::String_var repo_id = this->_repository_id ();
 
     if (TAO_debug_level > 5)
-      ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) _repository_id () -> %s\n", repo_id.in ()));
+      ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) _repository_id () -> %C\n", repo_id.in ()));
 
     CORBA::Any _any;
     _any <<= repo_id.in ();
@@ -700,13 +627,13 @@ void DSI_Servant::inner_invoke (R2CORBA_ServerRequest_ptr request)
   }
 }
 
-void DSI_Servant::invoke_DSI (R2CORBA_ServerRequest_ptr request)
+void DSI_Servant::invoke_DSI (CORBA::ServerRequest_ptr request)
 {
   if (TAO_debug_level > 5)
     ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) - Servant::invoke_DSI(%C) entry\n", request->operation ()));
 
   // wrap request for Ruby; cleanup automatically
-  ACE_Auto_Basic_Ptr<DSI_Data>  dsi_data(new DSI_Data(request));
+  std::unique_ptr<DSI_Data>  dsi_data(new DSI_Data(request));
 
   VALUE srvreq = Data_Wrap_Struct(r2tao_cServerRequest, 0, 0, dsi_data.get ());
 
@@ -752,7 +679,7 @@ void DSI_Servant::invoke_DSI (R2CORBA_ServerRequest_ptr request)
       _exc->completed (
         static_cast<CORBA::CompletionStatus> (NUM2ULONG (rb_iv_get (rexc, "@completed"))));
 
-      ACE_Auto_Basic_Ptr<CORBA::SystemException> e_ptr(_exc);
+      std::unique_ptr<CORBA::SystemException> e_ptr(_exc);
       _exc->_raise ();
     }
     else
@@ -825,7 +752,7 @@ void DSI_Servant::invoke_DSI (R2CORBA_ServerRequest_ptr request)
   }
 }
 
-void DSI_Servant::invoke_SI (R2CORBA_ServerRequest_ptr request)
+void DSI_Servant::invoke_SI (CORBA::ServerRequest_ptr request)
 {
   if (TAO_debug_level > 5)
     ACE_DEBUG ((LM_INFO, "R2TAO (%P|%t) - Servant::invoke_SI(%C) entry\n", request->operation ()));
@@ -985,7 +912,7 @@ void DSI_Servant::invoke_SI (R2CORBA_ServerRequest_ptr request)
         _exc->completed (
           static_cast<CORBA::CompletionStatus> (NUM2ULONG (rb_iv_get (rexc, "@completed"))));
 
-        ACE_Auto_Basic_Ptr<CORBA::SystemException> e_ptr(_exc);
+        std::unique_ptr<CORBA::SystemException> e_ptr(_exc);
         _exc->_raise ();
       }
       else
@@ -1097,7 +1024,7 @@ CORBA::Boolean DSI_Servant::_is_a (const char *logical_type_id)
   }
 }
 
-CORBA::Boolean DSI_Servant::_non_existent (void)
+CORBA::Boolean DSI_Servant::_non_existent ()
 {
   static R2TAO_RBFuncall FN_non_existent ("_non_existent?");
 
@@ -1113,7 +1040,7 @@ CORBA::Boolean DSI_Servant::_non_existent (void)
   }
 }
 
-CORBA::Object_ptr DSI_Servant::_get_component (void)
+CORBA::Object_ptr DSI_Servant::_get_component ()
 {
   static R2TAO_RBFuncall FN_get_component ("_get_component");
 
@@ -1131,7 +1058,7 @@ CORBA::Object_ptr DSI_Servant::_get_component (void)
   }
 }
 
-const char *DSI_Servant::_interface_repository_id (void) const
+const char *DSI_Servant::_interface_repository_id () const
 {
   static R2TAO_RBFuncall FN_repository_id ("_repository_id");
 
@@ -1179,7 +1106,7 @@ const char *DSI_Servant::_interface_repository_id (void) const
   }
 }
 
-char * DSI_Servant::_repository_id (void)
+char * DSI_Servant::_repository_id ()
 {
   return CORBA::string_dup (this->_interface_repository_id ());
 }
@@ -1243,7 +1170,7 @@ VALUE r2tao_Servant_default_POA(VALUE self)
 {
   R2TAO_TRY
   {
-    DSI_Servant* _servant = 0;
+    DSI_Servant* _servant {};
     if (DATA_PTR (self) == 0)
     {
       // create new C++ servant object
@@ -1269,7 +1196,7 @@ VALUE r2tao_Servant_this(VALUE self)
   R2TAO_TRY
   {
     bool _new_srv = false;
-    DSI_Servant* _servant = 0;
+    DSI_Servant* _servant {};
     if (DATA_PTR (self) == 0)
     {
       // create new C++ servant object
@@ -1321,9 +1248,8 @@ VALUE r2tao_Servant_this(VALUE self)
 static VALUE
 srv_alloc(VALUE klass)
 {
-  VALUE obj;
   // we start off without the C++ representation
-  obj = Data_Wrap_Struct(klass, 0, srv_free, 0);
+  VALUE obj = Data_Wrap_Struct(klass, nullptr, srv_free, nullptr);
   return obj;
 }
 

data/ext/librpoa/servant.h

@@ -10,50 +10,33 @@
 # Copyright (c) Remedy IT Expertise BV
 #------------------------------------------------------------------*/
 
-#if ((TAO_MAJOR_VERSION < 2) || (TAO_MAJOR_VERSION == 2 && TAO_MINOR_VERSION == 0 && TAO_MICRO_VERSION <= 1))
-# define RPOA_NEED_DSI_FIX  1
-
-# include "srvreq_fix.h"
-
-# define R2CORBA_ServerRequest  R2CORBA::ServerRequest
-# define R2CORBA_ServerRequest_ptr  R2CORBA::ServerRequest_ptr
-#else
-# define R2CORBA_ServerRequest  CORBA::ServerRequest
-# define R2CORBA_ServerRequest_ptr  CORBA::ServerRequest_ptr
-#endif
-
 //-------------------------------------------------------------------
 //  R2TAO Servant class
 //
 //===================================================================
-
 class DSI_Servant : public PortableServer::DynamicImplementation
 {
 public:
+  /// ctor
+  DSI_Servant () = delete;
   DSI_Servant (VALUE rbServant);
-  // ctor
-  virtual ~DSI_Servant ();
-  // dtor
+  ~DSI_Servant () override;
 
-  virtual void invoke (CORBA::ServerRequest_ptr request);
-      //ACE_THROW_SPEC ((CORBA::SystemException));
-#if RPOA_NEED_DSI_FIX
-  void invoke_fix (R2CORBA::ServerRequest_ptr request);
-#endif
+  void invoke (CORBA::ServerRequest_ptr request) override;
 
-  virtual CORBA::RepositoryId _primary_interface (
+  CORBA::RepositoryId _primary_interface (
       const PortableServer::ObjectId &oid,
-      PortableServer::POA_ptr poa);
+      PortableServer::POA_ptr poa) override;
 
-  virtual CORBA::Boolean _is_a (const char *logical_type_id);
+  CORBA::Boolean _is_a (const char *logical_type_id) override;
 
-  virtual CORBA::Boolean _non_existent (void);
+  CORBA::Boolean _non_existent () override;
 
-  //virtual CORBA::InterfaceDef_ptr _get_interface (void);
+  //virtual CORBA::InterfaceDef_ptr _get_interface ();
 
-  virtual CORBA::Object_ptr _get_component (void);
+  CORBA::Object_ptr _get_component () override;
 
-  virtual char * _repository_id (void);
+  char * _repository_id () override;
 
   enum METHOD {
     NONE,
@@ -73,12 +56,7 @@ public:
   void activate_servant ();
 
 protected:
-  virtual const char *_interface_repository_id (void) const;
-
-#if RPOA_NEED_DSI_FIX
-  /// Turns around and calls invoke.
-  virtual void _dispatch (TAO_ServerRequest &request, void *context);
-#endif
+  const char *_interface_repository_id () const override;
 
   void register_with_servant ();
 
@@ -87,26 +65,26 @@ protected:
 
   METHOD  method_id (const char* method);
 
-  virtual void inner_invoke (R2CORBA_ServerRequest_ptr request);
+  void inner_invoke (CORBA::ServerRequest_ptr request);
 
-  void invoke_SI (R2CORBA_ServerRequest_ptr request);
-  void invoke_DSI (R2CORBA_ServerRequest_ptr request);
+  void invoke_SI (CORBA::ServerRequest_ptr request);
+  void invoke_DSI (CORBA::ServerRequest_ptr request);
 
   static VALUE _invoke_implementation(VALUE args);
 
 private:
+  /// The Ruby Servant
   VALUE rbServant_;
-  // The Ruby Servant
 
   CORBA::String_var repo_id_;
 
   struct ThreadSafeArg
   {
     ThreadSafeArg (DSI_Servant* srv,
-                   R2CORBA_ServerRequest_ptr req)
+                   CORBA::ServerRequest_ptr req)
       : servant_(srv), request_(req) {}
     DSI_Servant* servant_;
-    R2CORBA_ServerRequest_ptr request_;
+    CORBA::ServerRequest_ptr request_;
   };
 
   static void* thread_safe_invoke (void * arg);

data/lib/corba/cbase/ORB.rb

@@ -41,12 +41,12 @@ module R2CORBA
           prop = nil
           a1, a2, a3 = args
           if Array === a1
-            raise ArgumentError, "Incorrect nr. of arguments; #{args.size}" if args.size>3
+            raise ArgumentError, "Incorrect nr. of arguments; #{args.size}" if args.size > 3
             argv = a1
             orb_id = (Hash === a2 ? nil : a2)
             prop = (Hash === a2 ? a2 : a3)
           elsif args.size == 1 || Hash === a2
-            raise ArgumentError, "Incorrect nr. of arguments; #{args.size}" if args.size>2
+            raise ArgumentError, "Incorrect nr. of arguments; #{args.size}" if args.size > 2
             orb_id = a1
             prop = a2
           else
@@ -54,7 +54,7 @@ module R2CORBA
           end
           raise ArgumentError, "Invalid argument #{prop.class}; expected Hash" unless prop.nil? || Hash === prop
           unless prop.nil?()
-            prop.inject(argv) {|a,(k,v)| a << k; a << v; a}
+            prop.inject(argv) {|a, (k, v)| a << k; a << v; a}
           end
           @@cached_orb = CORBA::Native::ORB.init(argv.collect {|a| a.to_s }.concat(@@_default_args), orb_id.nil?() ? nil : orb_id.to_s)
         end
@@ -98,11 +98,11 @@ module R2CORBA
         else
           @running ||= true
           raise CORBA::BAD_INV_ORDER.new('ORB has been shutdown', 0, CORBA::COMPLETED_NO) if @shutdown
-          while (timeout.nil? or timeout>0) and !@shutdown
+          while (timeout.nil? or timeout > 0) and !@shutdown
             to = timeout || 0.05
             f, to = self.work_pending(to)
             timeout = to unless timeout.nil?
-            if f and !@shutdown and (timeout.nil? or timeout>0)
+            if f and !@shutdown and (timeout.nil? or timeout > 0)
               to = timeout || 0.05
               to = self.perform_work(to)
               timeout = to unless timeout.nil?
@@ -144,17 +144,20 @@ module R2CORBA
           @q_ = []
           super
         end
+
         def push(vfklass)
           synchronize do
             @q_ << vfklass
           end
         end
+
         def process_all(&block)
           synchronize do
             @q_.each { |vf| block.call(vf) }
             @q_.clear
           end
         end
+
         def empty?()
           f = false
           synchronize do
@@ -186,22 +189,22 @@ module R2CORBA
    private
      @@sigreg = {}
      def CORBA.signal_numbers
-      ( [1, # HUP
-         2, # INT
-         3, # QUIT
-         5, # TRAP
-         6, # ABRT
-         10, # USR1
-         12, # USR2
-         13, # SIGPIPE
-         14, # ALRM
-         15, # TERM
-         17, # CHLD
-         18, # CONT
-         23, # URG
-         30, # PWR
-         31  # SYS
-        ]) & Signal.list.values
+      ([1, # HUP
+        2, # INT
+        3, # QUIT
+        5, # TRAP
+        6, # ABRT
+        10, # USR1
+        12, # USR2
+        13, # SIGPIPE
+        14, # ALRM
+        15, # TERM
+        17, # CHLD
+        18, # CONT
+        23, # URG
+        30, # PWR
+        31  # SYS
+       ]) & Signal.list.values
      end
 
      def CORBA.handled_signals
@@ -210,7 +213,7 @@ module R2CORBA
          sigcmd = Signal.trap(signum, 'DEFAULT')
          Signal.trap(signum, sigcmd)
          @@sigreg[signum] = sigcmd
-         if sigcmd.respond_to?(:call) or ['IGNORE','SIG_IGN','EXIT'].include?(sigcmd.to_s)
+         if sigcmd.respond_to?(:call) or ['IGNORE', 'SIG_IGN', 'EXIT'].include?(sigcmd.to_s)
            signum
          else
            nil
@@ -222,7 +225,7 @@ module R2CORBA
      def CORBA.handle_signal(signum)
        if @@sigreg.has_key?(signum)
          if @@sigreg[signum].respond_to?(:call)
-           if @@sigreg[signum].respond_to?(:parameters) && @@sigreg[signum].parameters.size>0
+           if @@sigreg[signum].respond_to?(:parameters) && @@sigreg[signum].parameters.size > 0
              @@sigreg[signum].call(signum)
            else
              @@sigreg[signum].call

data/lib/corba/cbase/Request.rb

@@ -14,17 +14,17 @@ module R2CORBA
 
     module Request
 
-      def add_in_arg(tc, val, nm='')
+      def add_in_arg(tc, val, nm = '')
         self._arguments << [nm, CORBA::ARG_IN, tc, val]
         self._arguments.size
       end
 
-      def add_out_arg(tc, nm='')
+      def add_out_arg(tc, nm = '')
         self._arguments << [nm, CORBA::ARG_OUT, tc]
         self._arguments.size
       end
 
-      def add_inout_arg(tc, val, nm='')
+      def add_inout_arg(tc, val, nm = '')
         self._arguments << [nm, CORBA::ARG_INOUT, tc, val]
         self._arguments.size
       end

data/lib/corba/cbase/Streams.rb

@@ -42,7 +42,7 @@ module R2CORBA
           when TK_DOUBLE
             read_double()
           when TK_LONGDOUBLE
-            raise CORBA::NO_IMPLEMENT.new('LongDouble not supported',0,CORBA::COMPLETED_NO)
+            raise CORBA::NO_IMPLEMENT.new('LongDouble not supported', 0, CORBA::COMPLETED_NO)
           when TK_FIXED
             read_fixed()
           when TK_CHAR
@@ -98,7 +98,7 @@ module R2CORBA
           when TK_DOUBLE
             write_double(value)
           when TK_LONGDOUBLE
-            raise CORBA::NO_IMPLEMENT.new('LongDouble not supported',0,CORBA::COMPLETED_NO)
+            raise CORBA::NO_IMPLEMENT.new('LongDouble not supported', 0, CORBA::COMPLETED_NO)
           when TK_FIXED
             write_fixed(value)
           when TK_CHAR

data/lib/corba/cbase/Typecode.rb

@@ -118,7 +118,7 @@ module R2CORBA
             content_tc = args.shift
             length = args
             raise ArgumentError, 'expected CORBA::TypeCode' unless content_tc.is_a?(CORBA::TypeCode)
-            if length.size>1
+            if length.size > 1
               this_len = length.shift
               content_tc = self.class.new(content_tc, *length)
             else
@@ -196,7 +196,7 @@ module R2CORBA
 
         protected
 
-        def _create_tc(id,name,modifier,base,members)
+        def _create_tc(id, name, modifier, base, members)
           begin
             CORBA::Native::TypeCode.create_tc(TK_VALUE, id.to_s, name.to_s,
                                               CORBA::VT_MODIFIERS[modifier],
@@ -213,7 +213,7 @@ module R2CORBA
 
         protected
 
-        def _create_tc(id,name,modifier,base,members)
+        def _create_tc(id, name, modifier, base, members)
           begin
             CORBA::Native::TypeCode.create_tc(TK_EVENT, id.to_s, name.to_s,
                                               CORBA::VT_MODIFIERS[modifier],
@@ -321,7 +321,7 @@ module R2CORBA
 
         protected
 
-        def _create_tc(id,name,members)
+        def _create_tc(id, name, members)
           begin
             CORBA::Native::TypeCode.create_tc(TK_STRUCT, id.to_s, name.to_s, members)
           rescue ::NativeException
@@ -335,7 +335,7 @@ module R2CORBA
 
         protected
 
-        def _create_tc(id,name,members)
+        def _create_tc(id, name, members)
           begin
             CORBA::Native::TypeCode.create_tc(TK_EXCEPT, id.to_s, name.to_s, members)
           rescue ::NativeException
@@ -356,7 +356,7 @@ module R2CORBA
             @members = []
             def_inx = @tc_.default_index
             @tc_.member_count.times do |i|
-              if def_inx < 0  || def_inx != i
+              if def_inx < 0 || def_inx != i
                 ml_ = @tc_.member_label(i)
               else
                 ml_ = :default