Skip to content

Commit

Permalink
fix: use (anonymous) for empty functionNames when generating stack in…
Browse files Browse the repository at this point in the history
…formation (#66)

closes #65
  • Loading branch information
Dirk Eismann committed Mar 16, 2022
1 parent 7523b2f commit 8e2d1f0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion packages/firebase-crashlytics/index.android.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ export class Crashlytics implements ICrashlytics {
StackTrace.fromError(error).then((stack) => {
const traceElements = Array.create('java.lang.StackTraceElement', stack.length);
stack.forEach((item, i) => {
traceElements[i] = new java.lang.StackTraceElement('', item.functionName, item.fileName, -1);
traceElements[i] = new java.lang.StackTraceElement('', item.functionName || '(anonymous)', item.fileName, -1);
});
const t = new java.lang.Throwable(error.message);
t.setStackTrace(traceElements);
Expand Down
2 changes: 1 addition & 1 deletion packages/firebase-crashlytics/index.ios.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ export class Crashlytics implements ICrashlytics {
StackTrace.fromError(error).then((stack) => {
const traceElements = [];
stack.forEach((item, i) => {
traceElements[i] = FIRStackFrame.stackFrameWithSymbolFileLine(item.functionName, item.fileName, item.lineNumber);
traceElements[i] = FIRStackFrame.stackFrameWithSymbolFileLine(item.functionName || '(anonymous)', item.fileName, item.lineNumber);
});
const e = FIRExceptionModel.exceptionModelWithNameReason('JavaScriptError', error.message);
this.native.recordExceptionModel(e);
Expand Down

0 comments on commit 8e2d1f0

Please sign in to comment.