Re: [PATCH v3 04/17] fix: utils: Fix the extract_list() utility function

[Kiizuha <riskimuhammmad1@gmail.com>]

On 20/12/2022 07.45, Ammar Faizi wrote:
> On 12/20/22 7:33 AM, Alviro Iskandar Setiawan wrote:
>> On Tue, Dec 20, 2022 at 6:57 AM Muhammad Rizki <kiizuha@gnuweeb.org> 
>> wrote:
>>> +def extract_list(key: str, thread: Message):
>>> +       try:
>>> +               people = thread.get(key.lower())
>>> +               if not people:
>>> +                       return []
>>> +       except:
>>>                  return []
>>>          return __extract_list(people)
>>
>> Why does this have to be a "try and except"?
>> And not something like:
>>
>>          k = key.lower()
>>          if k in thread:
>>                  return __extract_list(people)
>>          return []
> 
> I don't know much about Python, but doesn't that only
> work for dict? Or it can use `in` operator too?

Message class has its own `get()` function.

> 
> `thread` is an instance of Message class object here.
> 

That's right, but if you use it like `str(thread)` it will return a full 
email content like email headers and email body.