Skip to content

AutoHotkey: missing definitions#2400

Merged
RunDevelopment merged 3 commits into
PrismJS:masterfrom
joedf:patch-1
May 25, 2020
Merged

AutoHotkey: missing definitions#2400
RunDevelopment merged 3 commits into
PrismJS:masterfrom
joedf:patch-1

Conversation

@joedf

@joedf joedf commented May 23, 2020

Copy link
Copy Markdown
Contributor

Missing definitions

Thanks to ahk users BoBo and Xtra for reporting these.
https://www.autohotkey.com/boards/viewtopic.php?f=3&t=76304

constant

A_FileEncoding
A_Is64bitOS
A_PriorKey
A_PtrSize
A_RegView
A_ScreenDPI
A_ScriptHwnd

keyword

Throw
Try
Catch
Finally
Until
ahk_exe

builtin

LTrim() - was incorrect
RTrim()
StrReplace()
ComObjActive()
ComObjArray()
ComObjConnect()
ComObjCreate()
ComObjError()
ComObjFlags()
ComObjGet()
ComObjQuery()
ComObjType()
ComObjValue()

important

#DerefChar
#If
#IfTimeout
#InputLevel
#MenuMaskKey
#Warn

selector

SetRegView

# Missing definitions

constant
================
A_FileEncoding
A_Is64bitOS
A_PriorKey
A_PtrSize
A_RegView
A_ScreenDPI
A_ScriptHwnd

keyword
====================
Throw
Try
Catch
Finally
Until
ahk_exe

builtin
====================
LTrim() - was incorrect
RTrim()
StrReplace()
ComObjActive()
ComObjArray()
ComObjConnect()
ComObjCreate()
ComObjError()
ComObjFlags()
ComObjGet()
ComObjQuery()
ComObjType()
ComObjValue()

important
====================
#DerefChar
#If
#IfTimeout
#InputLevel
#MenuMaskKey
#Warn

selector
===================
SetRegView
@RunDevelopment

Copy link
Copy Markdown
Member

Thank you for making this PR @joedf!

Before we can merge this you need to rebuild Prism by running npx gulp.
Also, please add your changes to the corresponding test files. Here's how you edit test files.

@joedf

joedf commented May 25, 2020

Copy link
Copy Markdown
Contributor Author

Done! 👍

@RunDevelopment
RunDevelopment merged commit 4fe0367 into PrismJS:master May 25, 2020
@RunDevelopment

Copy link
Copy Markdown
Member

Thank you for contributing @joedf!

quentinvernot pushed a commit to TankerHQ/prismjs that referenced this pull request Sep 11, 2020
This adds a lot of missing keywords, builtins, constants, and other definitions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants