Modify

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#123 closed Feature Request (Rejected)

I have added two user functions to array.au3

Reported by: doug@… Owned by:
Milestone: Component: AutoIt
Version: Severity:
Keywords: Cc:

Description

Total - returns the total of all values in an array
Average - returns the average of all values in an array

Files are included with the additions.

Attachments (1)

au3.zip (14.9 KB) - added by dbkaynor 12 years ago.
Zip file with new array functions

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by dbkaynor

Zip file with new array functions

comment:1 Changed 12 years ago by Valik

Is this really something we need to add? I mean, the code is incredibly simple to do. There are also a few problems with the two functions. First, the code doesn't validate that the array is single-dimensioned. It just blindly assumes all arrays are one-dimensional. Second, it doesn't allow specifying the start index. Valid data may not start at element 0, it may start at element 1 depending on how the array was created. Third, there's an unused variable in one of the functions. Lastly, the documentation is wrong, it doesn't return an array.

All that being said, I really don't think we need to include these. Do a lot of people need these? Adding these seems like needless bloat to me. It kind of falls into the "far too simple" category. I leave it up to Gary to add or reject this, but my vote goes for reject.

comment:2 Changed 12 years ago by Gary

  • Resolution set to rejected
  • Status changed from new to closed

comment:3 Changed 12 years ago by Saunders <admin@…>

  • Resolution Rejected deleted
  • Status changed from closed to reopened

Is this even the proper way to submit UDFs? Forgetting for a moment that these would be rejected anyway, shouldn't he have gone through different channels?

comment:4 Changed 12 years ago by Saunders <admin@…>

Er, reopened??? Crap, that was not intentional, I don't know how I managed that, I don't remember clicking on that. Sorry guys.

comment:5 Changed 12 years ago by Valik

  • Resolution set to Rejected
  • Status changed from reopened to closed

You're right, it's really not the best way to go about it.

comment:6 Changed 12 years ago by TicketCleanup

  • Version 3.2.10.0 deleted

Automatic ticket cleanup.

Guidelines for posting comments:

  • You cannot re-open a ticket but you may still leave a comment if you have additional information to add.
  • In-depth discussions should take place on the forum.

For more information see the full version of the ticket guidelines here.

Add Comment

Modify Ticket

Action
as closed The ticket will remain with no owner.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.