Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bReFN-0007k4-TZ for pgadmin-hackers@arkaria.postgresql.org; Mon, 25 Jul 2016 11:40:34 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1bReFN-0004ft-GY for pgadmin-hackers@arkaria.postgresql.org; Mon, 25 Jul 2016 11:40:33 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1bReFA-0004OY-AP for pgadmin-hackers@postgresql.org; Mon, 25 Jul 2016 11:40:20 +0000 Received: from mail-io0-x22d.google.com ([2607:f8b0:4001:c06::22d]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.84_2) (envelope-from ) id 1bReF5-0005eE-RF for pgadmin-hackers@postgresql.org; Mon, 25 Jul 2016 11:40:19 +0000 Received: by mail-io0-x22d.google.com with SMTP id b62so158627740iod.3 for ; Mon, 25 Jul 2016 04:40:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pgadmin-org.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=mh9osK2reWo8GoGX2VVDeVKKpAB7+FDtCjrOyO0M8qY=; b=ZYOkjDsTi/ueUzuNXkTiQkhkHs3ZIknjmd7vajFODYdv05Y/xF19q5Hr8J1h+MGfyZ uKSm2xpr98tXEzrIi6O8GUvrVo+eqg+ibYQfaOBkXKY0NDxzXRpZWIEqJFQB6o+kM2l3 5M4Erc/nQ5VUvfVMQfLwBwkQCs7Lgoh7faPdwl1yCj9xPJ+VUJAUUaNnESe4QGGNOO0Z wf/WibUrNrFCVjXujbmsDg+xRt53yWOCRbOIZzl5yCxZpl/ciVa7D4btMEethPTBeZR2 eqB8neK3NdfJwLeLso8j2gmI4IG5t7yC44nV+bdLcarHNb+57twrLiSr62c7+mKiiLTE W0Yg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=mh9osK2reWo8GoGX2VVDeVKKpAB7+FDtCjrOyO0M8qY=; b=YbmufN31cK99xCJ/sUVCRhkLFdb5JyHCajNRVk+eAXho7gM25Qi9Fldk1K+P1eaGHN 4iODx3NCdzykBJGVDYAnuR8uaq/RcDpb799L1Hd4Caw4s2fC5YibmFczeQYdC79gf8AW IuMG+AVCgWsrbVy1TJYiXSfiDyynyJ0p188iMAkm48UfY7N9KWf+J9FDKkC0WxwG6xl5 cD7bATuxKdmY29for7Jcn5eH3UC3ozfEGFqZdhEu9bULMAaIUTK0Ov2+QGCCQl5TU1la smeOvvgBc06liSVPpEtjDSUu++ZPXC/rvkrREAP2trqUAcwYfBunWYyUnoLjyUH+NN6L +Shg== X-Gm-Message-State: AEkoouv7ys6ziP4x215kij+9wD1cjojpqRhocNNQwrA2Jqyg44AuGN7nONA9xhrIN3t8cruBQe4FT81+zP5DNQ== X-Received: by 10.107.160.204 with SMTP id j195mr20059328ioe.70.1469446814221; Mon, 25 Jul 2016 04:40:14 -0700 (PDT) MIME-Version: 1.0 Received: by 10.64.208.97 with HTTP; Mon, 25 Jul 2016 04:40:13 -0700 (PDT) In-Reply-To: References: From: Dave Page Date: Mon, 25 Jul 2016 12:40:13 +0100 Message-ID: Subject: Re: [pgAdmin4][runtime][patch]: Compilation error with Qt4 To: Neel Patel Cc: pgadmin-hackers Content-Type: text/plain; charset=UTF-8 X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgadmin-hackers Precedence: bulk Sender: pgadmin-hackers-owner@postgresql.org Thanks - applied. On Sat, Jul 23, 2016 at 2:39 PM, Neel Patel wrote: > Hi, > > Please find attached patch file with the fix of below issues with runtime > application. > > Compilation error with Qt4 as "QUrlQuery" class was introduced since Qt5. > Client side download code has been removed as now we support server side > download. > We should not delete sender object inside slot. If we do so it may crash the > application. To avoid this, we used Qt's "deleteLater" which will delete the > objects once the processing of all the events have been done. > > Do review it and let us know for comments. > > Thanks, > Neel Patel > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers