Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat/log #489

Merged
merged 5 commits into from
Aug 31, 2024
Merged

feat/log #489

merged 5 commits into from
Aug 31, 2024

Conversation

SuperFola
Copy link
Member

  • refactor(pass): moving the logging outside the Pass object to a proper Logger object
  • chore: replacing std::printf by std::cout
  • refactor: remove termcolor to use fmt color support

Copy link

github-actions bot commented Aug 31, 2024

Static analysis report

Lizard report

Listing only functions with cyclomatic complexity >= 15 or NLOC >= 100 or parameters >= 6.

Filename Start line:end line Function name Parameters NLOC CCN
src/arkreactor/VM/VM.cpp 266:1100 Ark::VM::safeRun 2 674 169
src/arkreactor/Compiler/BytecodeReader.cpp 196:518 Ark::BytecodeReader::display 4 289 125
src/arkreactor/Compiler/Macros/Processor.cpp 273:566 Ark::internal::MacroProcessor::evaluate 3 276 110
src/arkscript/REPL/Repl.cpp 17:225 Ark::Repl::Repl 1 176 35
src/arkreactor/Compiler/Compiler.cpp 558:688 Ark::Compiler::handleCalls 5 102 34
include/Ark/Compiler/AST/Parser.hpp 87:176 Ark::internal::ARK_APIParser::string 0 87 32
src/arkscript/main.cpp 17:320 main 2 262 28
src/arkreactor/Compiler/Compiler.cpp 265:356 Ark::Compiler::compileExpression 5 75 25
src/arkreactor/Compiler/AST/Parser.cpp 270:397 Ark::internal::Parser::import_ 0 103 21
src/arkreactor/Compiler/Macros/Processor.cpp 124:194 Ark::internal::MacroProcessor::processNode 2 54 21
src/arkreactor/Compiler/Macros/Processor.cpp 53:98 Ark::internal::MacroProcessor::registerMacro 1 41 16
src/arkreactor/Exceptions.cpp 86:130 Ark::Diagnostics::makeContext 6 35 15
src/arkreactor/TypeChecker.cpp 28:85 Ark::types::displayContract 2 51 15
src/arkscript/Formatter.cpp 152:205 Formatter::format 3 51 15
src/arkreactor/Exceptions.cpp 133:142 Ark::Diagnostics::helper 9 9 3
include/Ark/Exceptions.hpp 92:101 Ark::CodeError::CodeError 6 10 1
Report about files you didn't modify in this PR
Filename Start line:end line Function name Parameters NLOC CCN
src/arkreactor/Compiler/AST/Node.cpp 209:279 Ark::internal::Node::debugPrint 1 61 23
src/arkscript/JsonCompiler.cpp 24:189 JsonCompiler::_compile 1 139 23
include/utf8.hpp 138:184 utf8::isValid 1 44 21
src/arkreactor/Compiler/NameResolutionPass.cpp 130:207 Ark::internal::NameResolutionPass::visitKeyword 2 70 20
src/arkreactor/Compiler/Macros/Executors/Function.cpp 12:86 Ark::internal::FunctionExecutor::applyMacro 2 60 18
src/arkreactor/Compiler/AST/Node.cpp 143:207 Ark::internal::Node::repr 0 56 17
src/arkreactor/VM/Value.cpp 77:135 Ark::Value::toString 1 46 17
include/Ark/Compiler/AST/Predicates.hpp 162:186 Ark::internal::IsSymbol::operator ( ) 1 24 16

CppCheck report

Filename Line Type Description
src/arkreactor/Builtins/IO.cpp 24 style Parameter 'n' can be declared with const
src/arkreactor/Builtins/IO.cpp 43 style Parameter 'n' can be declared with const
src/arkreactor/Compiler/AST/Parser.cpp 389 error Uninitialized variable: import_data.with_prefix
src/arkreactor/Compiler/BytecodeReader.cpp 331 style struct member 'Arg::arg' is never used.
src/arkreactor/Compiler/Macros/Processor.cpp 27 performance Variable 'm_predefined_macros' is assigned in constructor body. Consider performing initialization in initialization list.
src/arkreactor/Compiler/Macros/Processor.cpp 252 performance Function parameter 'origin' should be passed by const reference.
src/arkscript/REPL/Repl.cpp 24 style Consider using std::copy algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 26 style Consider using std::copy algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 28 style Consider using std::copy algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 30 style Consider using std::copy algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 34 style Consider using std::transform algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 36 style Consider using std::transform algorithm instead of a raw loop.
src/arkscript/REPL/Repl.cpp 45 style Consider using std::transform algorithm instead of a raw loop.
src/arkscript/main.cpp 75 information Unmatched suppression: constStatement
Report files about files you didn't modify in this PR
Filename Line Type Description
include/Ark/Compiler/AST/Optimizer.hpp 64 style Unused private function: 'Optimizer::throwOptimizerError'
include/Ark/Compiler/AST/Optimizer.hpp 70 style Unused private function: 'Optimizer::removeUnused'
include/Ark/Compiler/Macros/Processor.hpp 165 performance Function parameter 'origin' should be passed by const reference.
src/arkreactor/Compiler/Macros/Executor.cpp 12 performance Function parameter 'origin' should be passed by const reference.
include/Ark/Compiler/Macros/Executor.hpp 65 performance Function parameter 'origin' should be passed by const reference.
src/arkreactor/Compiler/Macros/Executors/Function.cpp 14 style The scope of the variable 'first' can be reduced.
src/arkreactor/VM/Scope.cpp 51 style Variable 'id' is not assigned a value.
src/arkreactor/VM/Scope.cpp 61 style Variable 'id' is not assigned a value.
src/arkreactor/VM/Scope.cpp 71 style Variable 'id' is not assigned a value.

Copy link

github-actions bot commented Aug 31, 2024

Fuzzing report

/usr/local/bin/afl-whatsup status check tool for afl-fuzz by Michal Zalewski

Summary stats

    Fuzzers alive : 0
   Dead or remote : 1 (included in stats)
   Total run time : 5 minutes, 0 seconds
      Total execs : 43 thousands
 Cumulative speed : 146 execs/sec
    Pending items : 106 faves, 820 total
 Coverage reached : 10.56%
    Crashes saved : 0
      Hangs saved : 0

Cycles without finds : 0
Time without finds : 0

[+] Captured 30344 tuples (map size 172310, highest value 255, total values 262070045) in '/dev/null'.
[+] A coverage of 30344 edges were achieved out of 172352 existing (17.61%) with 828 input files.

@SuperFola SuperFola merged commit ff04fd5 into dev Aug 31, 2024
20 checks passed
@SuperFola SuperFola deleted the feat/log branch August 31, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant