fix makeAttachmentPointer to make 3 parameter optional

libtextsecure/sendmessage.js

@@ -193,11 +193,8 @@ MessageSender.prototype = {
   constructor: MessageSender,
 
   //  makeAttachmentPointer :: Attachment -> Promise AttachmentPointerProto
-  async makeAttachmentPointer(
+  async makeAttachmentPointer(attachment, publicServer = null, options = {}) {
-    attachment,
+    const { isRaw = false, isAvatar = false } = options;
-    publicServer = null,
-    { isRaw = false, isAvatar = false }
-  ) {
     if (typeof attachment !== 'object' || attachment == null) {
       return Promise.resolve(undefined);
     }