Should end method return a undefined?

[sosoba]

Method end has a specific case when it return undefined.

public end(raw: boolean = false) {
  // ...  
    const matches = dataBitsLen.toString(16).match(/(.*?)(.{0,8})$/);
      if (matches === null) {
        return;
       }
  // ...  
}
``

Maybe this case should raise an exception?

[anmiles]

That was breaking change. Previously worked code that uses this library was not build because it didn't expect undefined there.
I assume this should increase major version or this change should be reverted.

[sosoba]

Do you have data for which this case occurs? Maybe it's worth adding it in unit tests.

[anmiles]

const hash = new Md5().appendStr(str).end(false).toString()

Here in TS I need hash to exactly be a string. This was broken. However I temporarily replaced .toString() with as string.