OpenFilesScreen update

[cgzones]

• Remove local types and function from header file

• Reduce OpenFiles_Data to neccessary size

• Print file access mode (r/w/u)

• Fix memory leak on consecutive items without an intermediate file item:

  ==15257==ERROR: LeakSanitizer: detected memory leaks

  Direct leak of 120 byte(s) in 12 object(s) allocated from:
  #0 0x48c864 in strdup (htop/htop+0x48c864)
  Create ci.yml #1 0x542f68 in xStrdup htop/XAlloc.c:71:17
  fixed display of blank bars #2 0x50e225 in OpenFilesScreen_getProcessData htop/OpenFilesScreen.c:112:25
  Migrate previous releases #3 0x50cd17 in OpenFilesScreen_scan htop/OpenFilesScreen.c:141:35
  CI improvements #4 0x4fd3eb in InfoScreen_run htop/InfoScreen.c:81:35
  Fix build on FreeBSD #5 0x4d58bb in actionLsof htop/Action.c:361:4
  Switch to Python 3 #6 0x501766 in MainPanel_eventHandler htop/MainPanel.c:80:19
  Fix out-of-tree builds #7 0x5289fa in ScreenManager_run htop/ScreenManager.c:227:19
  CI: Add a HWLOC enabled job #8 0x4f748e in main htop/htop.c:300:4
  [Q] How to move forward with the generated header files? #9 0x7ff73e0d8cc9 in __libc_start_main csu/../csu/libc-start.c:308:16

  SUMMARY: AddressSanitizer: 120 byte(s) leaked in 12 allocation(s).

[BenBE]

LGTM.

Just a minor style note: In the long switch statement you could consider to just add a default: label to indicate all, not just the known columns, to be ignored.

[cgzones]

In the long switch statement you could consider to just add a default: label to indicate all, not just the known columns, to be ignored.

The body of that case statement is empty, so every compiler should drop it entirely; it's mainly a in-code documentation which fields are deliberately ignored.