Skip to content

Conversation

@rtakacs
Copy link
Contributor

@rtakacs rtakacs commented Jan 27, 2016

Avoid linking errors when building Jerry as a library (eg. for IoT.js).

@LaszloLango
Copy link
Contributor

LGTM

@LaszloLango LaszloLango added the bug Undesired behaviour label Jan 27, 2016
@zherczeg
Copy link
Member

zherczeg commented Feb 1, 2016

Please update this patch to the latest jerry.

@rtakacs
Copy link
Contributor Author

rtakacs commented Feb 1, 2016

Done.

* distributed under the License is distributed on an "AS IS" BASIS
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copyright years need to be updated.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

JerryScript-DCO-1.0-Signed-off-by: Roland Takacs rtakacs.u-szeged@partner.samsung.com
@rtakacs
Copy link
Contributor Author

rtakacs commented Feb 5, 2016

I have rebased the patch to the master.

@LaszloLango
Copy link
Contributor

@rtakacs, thanks. Still looks good to me.

@LaszloLango LaszloLango added the critical Raises security concerns label Feb 8, 2016
@LaszloLango
Copy link
Contributor

Landed (6608cc4)

@LaszloLango LaszloLango closed this Feb 9, 2016
@rtakacs rtakacs deleted the jerry-port-funcs branch January 18, 2019 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Undesired behaviour critical Raises security concerns

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants