Vyper: Rename event keyword

Created on 8 Mar 2018  路  4Comments  路  Source: vyperlang/vyper

This is the suggested template for new VIPs.

Preamble

VIP: 695
Title: Rename dunder log define function to event.
Author: Jacques Wagener 
Type: Standard Track 
Status: Draft
Created: 2018-03-08

Simple Summary

Rename __log__ keyword on definition of a event to event keyword instead.

Motivation

As a few users have pointed out, the __log__ keyword in defining an event is not as clear as using event({arg1: int128}) style defintions.

Specification

MyLog: __log__({arg1: indexed(bytes <= 3)})

Becomes

MyLog: event({arg1: indexed(bytes <= 3)})

The emitting of a log event will remain the same:

log.MyLog("123")

Backwards Compatibility

This is not backward compatible (full rename).

Copyright

Copyright and related rights waived via CC0

Approved

Most helpful comment

Yes I feel the event / log keyword combination will work well.

All 4 comments

Note: the syntax for "emitting"/logging an event is still log.MyLog('arg')

Yes I feel the event / log keyword combination will work well.

(put it in the issue lol)

Hey, I think we can close this issue now.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

domrany64 picture domrany64  路  3Comments

pipermerriam picture pipermerriam  路  3Comments

ben-kaufman picture ben-kaufman  路  3Comments

lsaether picture lsaether  路  4Comments

nrryuya picture nrryuya  路  3Comments