Skip to content

Commit cd73027

Browse files
yahondaclaude
andcommitted
Replace if not value.nil? with unless value.nil?
Two adjacent `if not value.nil?` trailing modifiers in `add_argument`'s XMLTYPE branch used the awkward double negation form. Replace with `unless value.nil?` for idiomatic Ruby; same semantics, same line count. No behavior change. Full suite is green (468 examples, 0 failures, 1 pre-existing pending). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent e12dd09 commit cd73027

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

lib/plsql/procedure_call.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -243,8 +243,8 @@ def add_argument(argument, value, argument_metadata = nil)
243243
when "UNDEFINED", "XMLTYPE", "OPAQUE/XMLTYPE"
244244
if xmltype_argument?(argument_metadata)
245245
@declare_sql << "l_#{argument} XMLTYPE;\n"
246-
@assignment_sql << "l_#{argument} := XMLTYPE(:#{argument});\n" if not value.nil?
247-
@bind_values[argument] = value if not value.nil?
246+
@assignment_sql << "l_#{argument} := XMLTYPE(:#{argument});\n" unless value.nil?
247+
@bind_values[argument] = value unless value.nil?
248248
@bind_metadata[argument] = argument_metadata.merge(data_type: "CLOB")
249249
"l_#{argument}"
250250
end

0 commit comments

Comments
 (0)