Opened 14 years ago
Last modified 13 years ago
#2199 closed Bug
_ExcelSheetAddNew() abends if specified sheet already exists — at Version 4
| Reported by: | Spiff59 | Owned by: | |
|---|---|---|---|
| Milestone: | 3.3.9.5 | Component: | Standard UDFs |
| Version: | 3.3.8.1 | Severity: | None |
| Keywords: | Cc: |
Description (last modified by )
As the Excel UDF is being brushed up this seems to be the right moment to reopen and solve this ticket.
Change History (4)
comment:1 by , 14 years ago
comment:2 by , 14 years ago
| Resolution: | → Rejected |
|---|---|
| Status: | new → closed |
That function doesn't follow its specification regarding return values.
If you want to submit some code then either do it correctly or don't do it at all. I understand the desire to be useful but please don't do it by adding new bugs.
I'll close this as "Rejected", and I'm hoping you will report it again, only correctly next time. By correctly I mean with explanation of the bug, reproducer (where is it now?), and suggestions if you have any.
comment:3 by , 14 years ago
Oh, the SetError() returns... shame on me.
Leaving it as rejected works for me.
comment:4 by , 13 years ago
| Component: | AutoIt → Standard UDFs |
|---|---|
| Description: | modified (diff) |
| Resolution: | Rejected |
| Status: | closed → reopened |
Note:
See TracTickets
for help on using tickets.
I suppose I could have made the @error = 2 test a single-line If/Then statement, if anyone cares...