You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by ngara <gi...@git.apache.org> on 2016/08/22 22:51:11 UTC

[GitHub] trafficserver pull request #893: TS-4581 CID 1356973 dead code in proxy/hdrs...

GitHub user ngara opened a pull request:

    https://github.com/apache/trafficserver/pull/893

    TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.cc

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ngara/trafficserver TS-4581

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/893.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #893
    
----
commit 6cc8fbd1b267324d86db17e1ff8e6a3ece7b2772
Author: Nathan Garabedian <ng...@apple.com>
Date:   2016-06-24T22:13:46Z

    TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.cc

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by ngara <gi...@git.apache.org>.
Github user ngara commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    ok, conflicts resolved again


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/589/ for details.
     



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    [approve ci]


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    Hmmm, still seeing branch merge conflicts :-/


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    I'm not 100% certain that Coverity will think this is enough, but lets try. @jpeach ok with this ?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/675/ for details.
     



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver pull request #893: TS-4581 CID 1356973 dead code in proxy/hdrs...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop closed the pull request at:

    https://github.com/apache/trafficserver/pull/893


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    [approve ci]


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/571/ for details.
     



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    Seems this branch has merge conflicts, please rebase with current master and push again before review.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/485/ for details.
     



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] trafficserver issue #893: TS-4581 CID 1356973 dead code in proxy/hdrs/HTTP.c...

Posted by ngara <gi...@git.apache.org>.
Github user ngara commented on the issue:

    https://github.com/apache/trafficserver/pull/893
  
    Please take a look again, it should be fixed now.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---