Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#1822 closed Bug (Wont Fix)

_GUICtrlListView_GetCounterPage???

Reported by: Matt Diesel Owned by: Gary
Milestone: Component: Standard UDFs
Version: 3.3.6.1 Severity: None
Keywords: Cc:

Description

Ummm... Is there a reason for the ridiculous naming of this function?

_GUICtrlListView_GetCount*P*erPage

It works fine (using send message with LVM_GETCOUNTPERPAGE) so it must be a typo, followed by a lot of people not noticing. It even says "ListView Get Counter Page" in the example.

Mat

Attachments (0)

Change History (7)

comment:1 by J-Paul Mesnage, 15 years ago

Resolution: No Bug
Status: newclosed

You right at was certainly a typo at the creation time 9/2007 but as it can lead to regression to existing script we don't plan to change it

in reply to:  1 ; comment:2 by Zedna, 15 years ago

Replying to Jpm:

You right at was certainly a typo at the creation time 9/2007 but as it can lead to regression to existing script we don't plan to change it

I disagree.
It should be fixed because there are many other script breaking changes
and similar typo in function name was already fixed:

http://www.autoitscript.com/trac/autoit/ticket/1015
Ticket #1015 (closed Bug: Fixed)
_SQLite_SaveMode() --> _SQLite_SafeMode()

in reply to:  2 comment:3 by J-Paul Mesnage, 15 years ago

Replying to Zedna:

Replying to Jpm:

You right at was certainly a typo at the creation time 9/2007 but as it can lead to regression to existing script we don't plan to change it

I disagree.
It should be fixed because there are many other script breaking changes
and similar typo in function name was already fixed:

http://www.autoitscript.com/trac/autoit/ticket/1015
Ticket #1015 (closed Bug: Fixed)
_SQLite_SaveMode() --> _SQLite_SafeMode()

on this #1015 I was the designer so I have taken the responsability of breaking . but on this one is not me.
Can you post on main forum an question about this breaking and report the conclusion if any?
Thanks.

PS I think a lot of UDF are never used ...

comment:4 by Matt Diesel, 15 years ago

I posted it on the forum here when I found it... I don't think it is used at all, as doing a search of the forum doesn't bring up any results other than a few people posting the name changes for the UDF between versions.

Even if it is left as is, there should at least be a note saying what it actually is, would have saved Kafu a bit of time by the looks of it :)

comment:5 by J-Paul Mesnage, 15 years ago

Resolution: No Bug
Status: closedreopened

comment:6 by J-Paul Mesnage, 15 years ago

Resolution: Wont Fix
Status: reopenedclosed

comment:7 by BrewmanNH, 15 years ago

In reading over this bug report, I'm wondering if there couldn't be a new function added to the LV UDF that uses the exact same function and just names it correctly?

This way, there's no script breaking changes made. Also in the help file you could either put a disclaimer in that mentions that the misspelled function is a deprecated feature and have a reference to the new _GUICtrlListView_GetCountPerPage function, or just leave both functions in the help file and hope no one notices that they're the same function. Just a thought that should make everyone happy, and make it easier for people to find the function if they're looking for a function that does what it does.

Modify Ticket

Action
as closed The owner will remain Gary.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.