summaryrefslogtreecommitdiffstats
path: root/Server/Plugins/APIDump/main_APIDump.lua
diff options
context:
space:
mode:
authorMattes D <github@xoft.cz>2016-09-13 09:43:02 +0200
committerGitHub <noreply@github.com>2016-09-13 09:43:02 +0200
commitca99e754ea430f500e79c5d929754c7e5946329b (patch)
tree5ea029a2a87d445f52ee80ca73c71e4bd37d3a0a /Server/Plugins/APIDump/main_APIDump.lua
parentBindings: Fixed some function signatures comments. (diff)
parentCIBuild: Mark as failure on duplicate API docs. (diff)
downloadcuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar.gz
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar.bz2
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar.lz
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar.xz
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.tar.zst
cuberite-ca99e754ea430f500e79c5d929754c7e5946329b.zip
Diffstat (limited to '')
-rw-r--r--Server/Plugins/APIDump/main_APIDump.lua142
1 files changed, 117 insertions, 25 deletions
diff --git a/Server/Plugins/APIDump/main_APIDump.lua b/Server/Plugins/APIDump/main_APIDump.lua
index 47848c26a..6dec8e159 100644
--- a/Server/Plugins/APIDump/main_APIDump.lua
+++ b/Server/Plugins/APIDump/main_APIDump.lua
@@ -36,6 +36,16 @@ local function LoadAPIFiles(a_Folder, a_DstTable)
break
end
for k, cls in pairs(Tables) do
+ if (a_DstTable[k]) then
+ -- The class is documented in two files, warn and store into a file (so that CIs can mark build as failure):
+ LOGWARNING(string.format(
+ "APIDump warning: class %s is documented at two places, the documentation in file %s will overwrite the previously loaded one!",
+ k, FileName
+ ))
+ local f = io.open("DuplicateDocs.txt", "a")
+ f:write(k, "\t", FileName)
+ f:close()
+ end
a_DstTable[k] = cls;
end
end -- if (TablesFn)
@@ -146,6 +156,12 @@ local function CreateAPITables()
end
end
+ -- Remove the built-in Lua libraries:
+ API.debug = nil
+ API.io = nil
+ API.string = nil
+ API.table = nil
+
return API, Globals;
end
@@ -505,8 +521,8 @@ local function ReadDescriptions(a_API, a_Desc)
local DoxyFunctions = {}; -- This will contain all the API functions together with their documentation
- local function AddFunction(a_Name, a_Params, a_Return, a_IsStatic, a_Notes)
- table.insert(DoxyFunctions, {Name = a_Name, Params = a_Params, Return = a_Return, IsStatic = a_IsStatic, Notes = a_Notes});
+ local function AddFunction(a_Name, a_Params, a_Returns, a_IsStatic, a_Notes)
+ table.insert(DoxyFunctions, {Name = a_Name, Params = a_Params, Returns = a_Returns, IsStatic = a_IsStatic, Notes = a_Notes});
end
if (APIDesc.Functions ~= nil) then
@@ -524,11 +540,11 @@ local function ReadDescriptions(a_API, a_Desc)
-- Description is available
if (FnDesc[1] == nil) then
-- Single function definition
- AddFunction(func.Name, FnDesc.Params, FnDesc.Return, FnDesc.IsStatic, FnDesc.Notes);
+ AddFunction(func.Name, FnDesc.Params, FnDesc.Returns, FnDesc.IsStatic, FnDesc.Notes);
else
-- Multiple function overloads
for _, desc in ipairs(FnDesc) do
- AddFunction(func.Name, desc.Params, desc.Return, desc.IsStatic, desc.Notes);
+ AddFunction(func.Name, desc.Params, desc.Returns, desc.IsStatic, desc.Notes);
end -- for k, desc - FnDesc[]
end
FnDesc.IsExported = true;
@@ -676,13 +692,6 @@ local function ReadDescriptions(a_API, a_Desc)
-- Sort the functions (they may have been renamed):
table.sort(cls.Functions,
function(f1, f2)
- if (f1.Name == f2.Name) then
- -- Same name, either comparing the same function to itself, or two overloads, in which case compare the params
- if ((f1.Params == nil) or (f2.Params == nil)) then
- return 0;
- end
- return (f1.Params < f2.Params);
- end
return (f1.Name < f2.Name);
end
);
@@ -761,7 +770,91 @@ end
-local function WriteHtmlClass(a_ClassAPI, a_ClassMenu)
+--- Returns a HTML string describing the (parameter) type, linking to the type's documentation, if available
+-- a_Type is the string containing the type (such as "cPlugin" or "number"), or nil
+-- a_API is the complete API description (used for searching the classnames)
+local function LinkifyType(a_Type, a_API)
+ -- Check params:
+ assert(type(a_Type) == "string")
+ assert(type(a_API) == "table")
+
+ -- If the type is a known class, return a direct link to it:
+ if (a_API[a_Type]) then
+ return "<a href=\"" .. a_Type .. ".html\">" .. a_Type .. "</a>"
+ end
+
+ -- If the type has a hash sign, it's a child enum of a class:
+ local idxColon = a_Type:find("#")
+ if (idxColon) then
+ local classType = a_Type:sub(1, idxColon - 1)
+ if (a_API[classType]) then
+ local enumType = a_Type:sub(idxColon + 1)
+ return "<a href=\"" .. classType .. ".html#" .. enumType .. "\">" .. enumType .. "</a>"
+ end
+ end
+
+ -- If the type is a ConstantGroup within the Globals, it's a global enum:
+ if ((a_API.Globals.ConstantGroups or {})[a_Type]) then
+ return "<a href=\"Globals.html#" .. a_Type .. "\">" .. a_Type .. "</a>"
+ end
+
+ -- Unknown or built-in type, output just text:
+ return a_Type
+end
+
+
+
+
+
+--- Returns an HTML string describing all function parameters (or return values)
+-- a_FnParams is an array-table or string description of the parameters
+-- a_ClassName is the name of the class for which the function is being documented (for Linkification)
+-- a_API is the complete API description (for cross-type linkification)
+local function CreateFunctionParamsDescription(a_FnParams, a_ClassName, a_API)
+ local pt = type(a_FnParams)
+ assert((pt == "string") or (pt == "table"))
+ assert(type(a_ClassName) == "string")
+ assert(type(a_API) == "table")
+
+ -- If the params description is a string (old format), just linkify it:
+ if (pt == "string") then
+ return LinkifyString(a_FnParams, a_ClassName)
+ end
+
+ -- If the params description is an empty table, give no description at all:
+ if not(a_FnParams[1]) then
+ return ""
+ end
+
+ -- The params description is a table, output the full desc:
+ local res = {"<table border=0 cellspacing=0>"}
+ local idx = 2
+ for _, param in ipairs(a_FnParams) do
+ res[idx] = "<tr><td>"
+ res[idx + 1] = param.Name or ""
+ res[idx + 2] = "</td><td><i>"
+ res[idx + 3] = LinkifyType(param.Type, a_API)
+ res[idx + 4] = "</i></td></tr>"
+ idx = idx + 5
+ end
+ res[idx] = "</tr></table>"
+ return table.concat(res)
+end
+
+
+
+
+
+--- Writes an HTML file containing the class API description for the given class
+-- a_ClassAPI is the API description of the class to output
+-- a_ClassMenu is the HTML string containing the code for the menu sidebar
+-- a_API is the complete API (for cross-type linkification)
+local function WriteHtmlClass(a_ClassAPI, a_ClassMenu, a_API)
+ -- Check params:
+ assert(type(a_ClassAPI) == "table")
+ assert(type(a_ClassMenu) == "string")
+ assert(type(a_API) == "table")
+
local cf, err = io.open("API/" .. a_ClassAPI.Name .. ".html", "w");
if (cf == nil) then
LOGINFO("Cannot write HTML API for class " .. a_ClassAPI.Name .. ": " .. err)
@@ -770,11 +863,12 @@ local function WriteHtmlClass(a_ClassAPI, a_ClassMenu)
-- Writes a table containing all functions in the specified list, with an optional "inherited from" header when a_InheritedName is valid
local function WriteFunctions(a_Functions, a_InheritedName)
- if (#a_Functions == 0) then
+ if not(a_Functions[1]) then
+ -- No functions to write
return;
end
- if (a_InheritedName ~= nil) then
+ if (a_InheritedName) then
cf:write("<h2>Functions inherited from ", a_InheritedName, "</h2>\n");
end
cf:write("<table>\n<tr><th>Name</th><th>Parameters</th><th>Return value</th><th>Notes</th></tr>\n");
@@ -789,8 +883,8 @@ local function WriteHtmlClass(a_ClassAPI, a_ClassMenu)
TableOverloadedFunctions[func.Name] = (TableOverloadedFunctions[func.Name] or 0) + 1
-- Add the anchor names as a title
cf:write("<tr><td id=\"", func.Name, "_", TableOverloadedFunctions[func.Name], "\" title=\"", func.Name, "_", TableOverloadedFunctions[func.Name], "\">", func.Name, "</td>\n");
- cf:write("<td>", LinkifyString(func.Params or "", (a_InheritedName or a_ClassAPI.Name)), "</td>\n");
- cf:write("<td>", LinkifyString(func.Return or "", (a_InheritedName or a_ClassAPI.Name)), "</td>\n");
+ cf:write("<td>", CreateFunctionParamsDescription(func.Params or {}, a_InheritedName or a_ClassAPI.Name, a_API), "</td>\n");
+ cf:write("<td>", CreateFunctionParamsDescription(func.Returns or {}, a_InheritedName or a_ClassAPI.Name, a_API), "</td>\n");
cf:write("<td>", StaticClause .. LinkifyString(func.Notes or "<i>(undocumented)</i>", (a_InheritedName or a_ClassAPI.Name)), "</td></tr>\n");
end
cf:write("</table>\n");
@@ -1032,7 +1126,7 @@ local function WriteClasses(f, a_API, a_ClassMenu)
]]);
for _, cls in ipairs(a_API) do
f:write("<li><a href=\"", cls.Name, ".html\">", cls.Name, "</a></li>\n");
- WriteHtmlClass(cls, a_ClassMenu);
+ WriteHtmlClass(cls, a_ClassMenu, a_API);
end
f:write([[
</ul></p>
@@ -1452,6 +1546,7 @@ local function WriteZBSMethods(f, a_Methods)
f:write("\t\t\t[\"", func.Name, "\"] =\n")
f:write("\t\t\t{\n")
f:write("\t\t\t\ttype = \"method\",\n")
+ -- No way to indicate multiple signatures to ZBS, so don't output any params at all
if ((func.Notes ~= nil) and (func.Notes ~= "")) then
f:write("\t\t\t\tdescription = [[", CleanUpDescription(func.Notes or ""), " ]],\n")
end
@@ -1648,14 +1743,10 @@ local function PrepareApi()
-- Load the API descriptions from the Classes and Hooks subfolders:
-- This needs to be done each time the command is invoked because the export modifies the tables' contents
local apiDesc = dofile(g_PluginFolder .. "/APIDesc.lua")
- if (apiDesc.Classes == nil) then
- apiDesc.Classes = {};
- end
- if (apiDesc.Hooks == nil) then
- apiDesc.Hooks = {};
- end
- LoadAPIFiles("/Classes/", apiDesc.Classes);
- LoadAPIFiles("/Hooks/", apiDesc.Hooks);
+ apiDesc.Classes = apiDesc.Classes or {}
+ apiDesc.Hooks = apiDesc.Hooks or {}
+ LoadAPIFiles("/Classes/", apiDesc.Classes)
+ LoadAPIFiles("/Hooks/", apiDesc.Hooks)
-- Reset the stats:
g_TrackedPages = {}; -- List of tracked pages, to be checked later whether they exist. Each item is an array of referring pagenames.
@@ -1689,6 +1780,7 @@ local function PrepareApi()
-- Add Globals into the API:
Globals.Name = "Globals";
table.insert(API, Globals);
+ API.Globals = Globals
-- Read in the descriptions:
LOG("Reading descriptions...");