Back to all reviewers

consistent naming conventions

hyprwm/Hyprland
Based on 6 comments
Other

Follow the project's established naming conventions consistently across all code. Use camelCase for variables, functions, and parameters, avoiding Hungarian notation and snake_case. Apply type-specific prefixes: classes should have a 'C' prefix (e.g., `CXCBConnection`), enums should have an 'e' prefix (e.g., `eAutoDirs`), and enum values should be in CAPS...

Naming Conventions Other

Reviewer Prompt

Follow the project’s established naming conventions consistently across all code. Use camelCase for variables, functions, and parameters, avoiding Hungarian notation and snake_case. Apply type-specific prefixes: classes should have a ‘C’ prefix (e.g., CXCBConnection), enums should have an ‘e’ prefix (e.g., eAutoDirs), and enum values should be in CAPS with descriptive prefixes (e.g., DIR_AUTO_UP). When possible, wrap standalone functions in appropriate namespaces rather than leaving them in global scope.

Example transformations:

// Before
class XCBConnection { };           // Missing C prefix
enum class AutoDirs { auto_up };   // Missing e prefix, snake_case
bool m_benabled;                   // Hungarian notation
void some_function();              // snake_case

// After  
class CXCBConnection { };          // C prefix for classes
enum class eAutoDirs { DIR_AUTO_UP }; // e prefix, CAPS enum values
bool m_enabled;                    // No Hungarian notation
void someFunction();               // camelCase

This ensures code readability and maintains consistency with the existing codebase, making it easier for developers to understand type information at a glance and follow established patterns.

6
Comments Analyzed
Other
Primary Language
Naming Conventions
Category

Source Discussions