Opened 7 years ago
Closed 3 years ago
#3538 closed Feature Request (Fixed)
Make ByRef parameters for _PathSplit optional
| Reported by: | TheDcoder <TheDcoder@…> | Owned by: | Melba23 |
|---|---|---|---|
| Milestone: | Component: | Standard UDFs | |
| Version: | Severity: | None | |
| Keywords: | Cc: |
Description
Hello, I think it will be the best if we make the ByRef parameters in the _PathSplit function optional because not everyone uses them :)
Attachments (0)
Change History (9)
comment:1 follow-up: ↓ 2 Changed 7 years ago by Melba23
comment:2 in reply to: ↑ 1 Changed 7 years ago by TheDcoder <TheDcoder@…>
No M23, I am not asking to remove the ByRef parameters, but to make them optional so we can do this: _PathSplit(@ScriptFullPath). I hope that you understand :)
comment:3 Changed 7 years ago by Melba23
I understand, but all my testing shows that you cannot have optional ByRef parameters - which is quite reasonable seeing as the whole point is to alter the external parameter from within the function.
M23
comment:4 Changed 7 years ago by TheDcoder <TheDcoder@…>
Ah! I just remembered that we cannot have optional ByRef parameters :(
I think I have found another "thing" which is very frustrating, this code won't work because of AU3Check's error: _PathSplit(@ScriptFullPath, "", "", "", "")
Am I in a situation where I must be using some dummy variables for those 4 parameters? :-/
comment:5 Changed 7 years ago by TheDcoder <TheDcoder@…>
I have just opened Ticket #3540: "Implement optional ByRef parameters for functions"
I thought it might relevant to reference it here :)
comment:6 Changed 4 years ago by Jpm
- Owner set to Melba23
- Status changed from new to assigned
comment:7 Changed 4 years ago by Melba23
- Resolution set to Rejected
- Status changed from assigned to closed
comment:8 Changed 3 years ago by mLipok
- Resolution Rejected deleted
- Status changed from closed to reopened
comment:9 Changed 3 years ago by mLipok
- Resolution set to Fixed
- Status changed from reopened to closed
I asked @jpm in PM on the forum about this:
https://www.autoitscript.com/trac/autoit/ticket/3540#comment:5
Answer: Should be fixed together with #3540
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.
I disagree.
If you look at the function in detail it both returns an array containing the path elements and sets the ByRef parameters to the same values. So removing the second of these functionalities could be script-breaking.
M23