Issue916

Title Support K,M,G suffixes for integer options in the code
Priority wish Status resolved
Superseder Nosy List jendrik, malte
Assigned To jendrik Keywords
Optional summary

Created on 2019-06-04.09:06:14 by jendrik, last changed by jendrik.

Messages
msg8814 (view) Author: jendrik Date: 2019-06-05.16:08:38
We talked about this offline, I added a comment to the code and merged it.
msg8794 (view) Author: jendrik Date: 2019-06-04.09:45:13
I think the pull request will make it clearer:
https://bitbucket.org/jendrikseipp/downward/pull-requests/134

Previously, we could not use "1M" in the code, only on the commandline.
msg8793 (view) Author: malte Date: 2019-06-04.09:12:04
I don't understand what you mean by this. Can you elaborate? Using the suffixes
seems to work fine.
msg8792 (view) Author: jendrik Date: 2019-06-04.09:06:14
In issue886 we added support for K,M,G suffixes for integer options in
commandline options. However, we forgot to allow using these suffixes in the code.
History
Date User Action Args
2019-06-05 16:08:38jendriksetstatus: reviewing -> resolved
messages: + msg8814
2019-06-04 09:45:23jendriksetstatus: chatting -> reviewing
assignedto: jendrik
2019-06-04 09:45:13jendriksetmessages: + msg8794
2019-06-04 09:12:04maltesetstatus: unread -> chatting
messages: + msg8793
2019-06-04 09:06:14jendrikcreate