[Lldb-commits] [lldb] [lldb] Adds additional fields to ProcessInfo (PR #91544)

[Greg Clayton via lldb-commits]

================
@@ -117,6 +118,10 @@ bool ProcessInfo::IsScriptedProcess() const {
   return m_scripted_metadata_sp && *m_scripted_metadata_sp;
 }
 
+bool ProcessInstanceInfo::NiceValueIsValid() const {
+  return m_nice_value >= PRIO_MIN && m_nice_value <= PRIO_MAX;
----------------
clayborg wrote:

`PRIO_MIN` and `PRIO_MAX` come from linux/unix header and won't be available on other systems. I am also wondering if the `PRIO_MIN` and `PRIO_MAX` will differ from one linux OS to another? Probably best to not rely on these defines from OS specific header files, so I would suggest either manually defining them in this file or not checking the values.

https://github.com/llvm/llvm-project/pull/91544