Ticket #6 (closed enhancement: fixed)

Opened 13 months ago

Last modified 13 months ago

Patch for the method add_field

Reported by: stephane.wirtel@… Owned by: segfault
Priority: major Milestone: Version 0.3
Component: HL7 Library Version: 0.2
Keywords: Cc:

Description

With this patch, you can put the name of the field as the first argument of the add_field method.

You win in readability, and you write less code

Attachments

add_name_arg_to_add_field_method.patch (23.6 kB) - added by stephane.wirtel@… 13 months ago.
add_name_arg_to_add_field_method_and_refactor_the_source_code.patch (25.2 kB) - added by stephane.wirtel@… 13 months ago.
New version of the previous patch

Change History

Changed 13 months ago by stephane.wirtel@…

  Changed 13 months ago by anonymous

  • version changed from 0.3 to 0.2

follow-up: ↓ 3   Changed 13 months ago by segfault

Looks great, I'll apply it later today.

in reply to: ↑ 2   Changed 13 months ago by anonymous

Replying to segfault:

Looks great, I'll apply it later today.

Thank you

Changed 13 months ago by stephane.wirtel@…

New version of the previous patch

  Changed 13 months ago by anonymous

I put a new patch with the previous patch. I refactor a small piece of code.

Thanks

  Changed 13 months ago by segfault

  • status changed from new to closed
  • resolution set to fixed

applied patch in [56]

Add/Change #6 (Patch for the method add_field)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.