[clang-format] Main headers are ignored if priority is negative

[EugeneZelenko]

My work project coding style convention requires main header file to be last one. It means that all other headers must have negative priorities. But IncludeCategoryManager::getIncludePriority and IncludeCategoryManager::getSortIncludePriority ignore such headers.

I tried 19.1.1 and main (49d48c3)

Configuration:

IncludeCategories:
  # C-Python header
  - Regex:           '^<Python\.h>$'
    Priority:        -6
    CaseSensitive:   true
  # System headers
  - Regex:           '^<.+\.h>$'
    Priority:        -5
    CaseSensitive:   true
  # STL headers
  - Regex:           '^<[a-z][a-z_]+>$'
    Priority:        -4
    CaseSensitive:   true
  # Boost headers
  - Regex:           '^<boost/.+>$'
    Priority:        -3
    CaseSensitive:   true
  # Qt headers
  - Regex:           '^<Q.+>$'
    Priority:        -2
    CaseSensitive:   true
  # Local headers
  - Regex:           '^".+"$'
    Priority:        -1
    CaseSensitive:   true

[llvmbot]

@llvm/issue-subscribers-clang-format

Author: None (EugeneZelenko)

My work project coding style convention requires main header file to be last one. It means that all other headers must have negative priorities. But `IncludeCategoryManager::getIncludePriority` and `IncludeCategoryManager::getSortIncludePriority` ignore such headers.

I tried 19.1.1 and main (49d48c3)

Configuration:

IncludeCategories:
  # C-Python header
  - Regex:           '^<Python\.h>$'
    Priority:        -6
    CaseSensitive:   true
  # System headers
  - Regex:           '^<.+\.h>$'
    Priority:        -5
    CaseSensitive:   true
  # STL headers
  - Regex:           '^<[a-z][a-z_]+>$'
    Priority:        -4
    CaseSensitive:   true
  # Boost headers
  - Regex:           '^<boost/.+>$'
    Priority:        -3
    CaseSensitive:   true
  # Qt headers
  - Regex:           '^<Q.+>$'
    Priority:        -2
    CaseSensitive:   true
  # Local headers
  - Regex:           '^".+"$'
    Priority:        -1
    CaseSensitive:   true

[owenca]

What's the input, output, and expected output?

[EugeneZelenko]

Test case (test.cpp):

#include "test/a.h"
#include "test/test.h"
#include "test/z.h"
#include <QObject>
#include <string>
#include <unistd.h>

Output:

#include <unistd.h>
#include <string>
#include <QObject>
#include "test/a.h"
#include "test/test.h"
#include "test/z.h"

Expected output:

#include <unistd.h>
#include <string>
#include <QObject>
#include "test/a.h"
#include "test/z.h"
#include "test/test.h"