Mantis Bug Tracker

View Issue Details Jump to Notes ] Wiki ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000082mSLDevAutoCompletepublic2013-03-23 12:002013-04-13 08:06
ReporterWims 
Assigned Toadministrator 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformWin7 64-bitOSWindows 7OS Version(6.1)
Product Version0.3.1.0 
Target VersionFixed in Version0.4.1.0 
Summary0000082: $nick()
DescriptionI typed '$nick(', msldev autocompletes with '$nick()' and then display in the error list: Ident $nick: too few parameters.
Steps To ReproduceType '$nick('
Additional InformationClearly, the $nick() function isn't available for calltip.
Now, the autocompletion is inconsistent:

-You type: '$bytes(' manually
>it auto adds the ), and put the cursor in the middle, this is actually the case and is fine.

-You type: '$by' and autocomplete $bytes
> it just write $bytes for you, without adding the ( and the ) and putting the cursor in the middle, no calltip either, this is wrong imo.

now, getting '$bytes()' to be displayed and moving the cursor somewhere else doesn't bring an error $bytes: too few parameter, why is so special about $nick?

Anyway this is several issues related together so i made only one post.
Since i'm here, the calltip should disappear automatically in some cases but doesn't, i'll make a seperate report.
TagsNo tags attached.
Beta mIRC version?
mIRC Version7.29
Attached Files

- Relationships

-  Notes
(0000037)
administrator (administrator)
2013-03-23 12:41
edited on: 2013-03-23 12:43

The auto-complete is consistent - nothing to do with the (->) completion.

When you type ( at the end of an identifier, mSLDev will insert a ). That's the only time.

Nothing is special about $nick per se. $bytes() can take 1 argument - no errors. $nick() cannot take only 1 argument, it's an error - that's why mSLDev is issuing an error.

(0000040)
Wims (reporter)
2013-03-23 13:34

Well, i'm saying when you autocomplete an identifier, it should get you to the calltip, with () already there, not just complete the name. That would be better, you're not going to complete it not to go ahead with it.

Now, the problem about reporting error is that it's not correct to yield at the user if you, msldev, actually completed the $identifier syntax (adding the ')') for the user. I suggest not reporting the error if the cursor is inside the $identifier(), that way you'll only report the error when the user keep going with the code, and is outsite that identifier, if he jumps back into the identifiers causing an error, you obviously keep the error, but if he fixes it and reintroduce a new error, you still wait for the cursor to be elsewhere.
(0000084)
administrator (administrator)
2013-04-13 08:06

Auto-complete now adds the () when appropriate. As for the errors showing, this behavior will remain for the time being. (All settings can be disabled via the option window).

- Issue History
Date Modified Username Field Change
2013-03-23 12:00 Wims New Issue
2013-03-23 12:41 administrator Note Added: 0000037
2013-03-23 12:43 administrator Note Edited: 0000037 View Revisions
2013-03-23 13:34 Wims Note Added: 0000040
2013-04-13 08:06 administrator Note Added: 0000084
2013-04-13 08:06 administrator Status new => closed
2013-04-13 08:06 administrator Assigned To => administrator
2013-04-13 08:06 administrator Resolution open => fixed
2013-04-13 08:06 administrator Fixed in Version => 0.4.1.0


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker