Modify

#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 Jpm, on Dec 10, 2010 at 11:58:41 AM

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, on Dec 10, 2010 at 11:34:55 PM

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 Jpm, on Dec 11, 2010 at 9:31:12 AM

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, on Dec 12, 2010 at 7:08:35 PM

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 Jpm, on Dec 19, 2010 at 10:45:51 AM

Resolution: No Bug
Status: closedreopened

comment:6 by Jpm, on Dec 19, 2010 at 10:46:03 AM

Resolution: Wont Fix
Status: reopenedclosed

comment:7 by BrewmanNH, on Feb 10, 2011 at 7:52:29 PM

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.