Skip to content

Commit

Permalink
Move all functions associated with display_cimobjects to separate
Browse files Browse the repository at this point in the history
module.

This is first part of adding the code for issue #249, display classes
as tables.  Because the new code is going to significantly increase the
size of the functions associated with displaying objects as tables, it
was logical to move this from _common.py to its own file.

Fixes alse  one pylint issue by changing code

Fixes issue where pylint was reporting possible undefined variable in
pick_one_from_list() when the variable was part of a for statement by
not using that variable and creating a new variable to represent the
same information.

In the process we noted that there was no test for the correct pick of
last item in the list and confirmation that if the next higher number
was picked it treated as invalid a so test was added.
  • Loading branch information
KSchopmeyer committed Aug 29, 2020
1 parent 6f7cbbf commit acc0d06
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pywbemtools/pywbemcli/_display_cimobjects.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@

INT_TYPE_PATTERN = re.compile(r'^[su]int(8|16|32|64)$')


####################################################################
#
# Display of CIM objects.
Expand Down

0 comments on commit acc0d06

Please sign in to comment.