protobuf icon indicating copy to clipboard operation
protobuf copied to clipboard

Generated code / formatting: separate parameters from local variables

Open zertovitch opened this issue 7 months ago • 0 comments

Please ignore if this detail is too difficult to implement.

When I read this kind of bodies:

   procedure Read_Empty
    (Stream : access Ada.Streams.Root_Stream_Type'Class;
     V      : out Empty) is
      Key : aliased PB_Support.IO.Key;
   begin

it takes me a while to notice that "Key" is not a (misaligned) parameter but a local variable. I prefer:

   procedure Read_Empty
    (Stream : access Ada.Streams.Root_Stream_Type'Class;
     V      : out Empty)
   is
      Key : aliased PB_Support.IO.Key;
   begin

zertovitch avatar Jun 18 '25 18:06 zertovitch