[Ethereum] Defining constructors as functions with the same name as the contract is deprecated. Use “constructor(…) { … }” instead

The problem is it doesn't compile.

constructor. This is wrong. In solc 0.4.22 you can say function constructor() but is new (today) so I used the more traditional "same name as contract" method.

tx.orgin is a security risk, so changed to msg.sender with a pass-through pattern.

pragma solidity ^0.4.17;

contract C {

  address public owner; 
  address public creator;

  function C(address _creator, address _owner) public {
    owner = _owner;
    creator = _creator;
  }

  function() public payable {}

  function printOwner() public view returns(address h) {
      h = owner;
  }

}

//factory contract for C
contract CFactory {
  address public owner;
  address public currentContractAddress;

  function CFactory() public {
    owner = msg.sender;
    currentContractAddress = address(this);
  }

  //function() public payable {}

  function test1() public returns(address){
    C c = new C(currentContractAddress, msg.sender);
    return c.printOwner();
  }

}

Hope it helps.

That is an outdated error message. I assume you are using ethlint as linter which means you can silence this warning by creating a .soliumrc.json in your repository with the following content:

{
  "extends": "solium:recommended",
  "plugins": ["security"],
  "rules": {
    "no-call-value": "off"
  }
}

See the Github issue and further explanations on how to use call.value.