Ticket #1952 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

t/src/misc.t tests 1 and 2 fail under C89 compiler

Reported by: doughera Owned by: cotto
Priority: normal Milestone:
Component: testing Version: 2.11.0
Severity: medium Keywords:
Cc: Language:
Patch status: applied Platform:

Description

t/src/misc.t tests 1 and 2 mix variable declarations and code. The attached simple patch moves the 'buf' declarations earlier in the function so that a C89 compiler will compile the tests correctly.

Attachments

tt1952.patch Download (0.8 KB) - added by doughera 3 years ago.

Change History

Changed 3 years ago by doughera

  Changed 3 years ago by doughera

  • patch set to new

  Changed 3 years ago by cotto

applied in be8f1a6. Thanks!

follow-up: ↓ 4   Changed 3 years ago by jkeenan

  • owner set to cotto
  • component changed from none to testing
  • patch changed from new to applied

Patch applied:

commit be8f1a6 | cotto++ | t/src/misc.t: [t] fix src tests for C89 compatibility; doughera++ for the patch

Andy, can you re-test?

Thanks.

kid51

in reply to: ↑ 3   Changed 3 years ago by doughera

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

Replying to jkeenan:

Patch applied:

Thanks. Closing ticket.

Note: See TracTickets for help on using tickets.