Build History for pull/540

Add changes entry Signed-off-by: Patrick Ferris <patrick@sirref.org>

Change Location.none Location.none has been changed to match the compiler's Location.none as of OCaml 4.08. This has been stable for some time. We also provide a Location.is_none which checks if a location is equal to Location.none for all versions of the compiler we currently support (i.e. >= 4.04). The decision to change Location.none has been made to enable the code to be simplified if we decide to change the lower bound of ppxlib to 4.08 in the future. Location.is_none has been used to fix loc_of_attribute which would sometimes miss attributes with none locations (like compiler inserted ocaml.doc attributes). Signed-off-by: Patrick Ferris <patrick@sirref.org>

Change Location.none Location.none has been changed to match the compiler's Location.none as of OCaml 4.08. This has been stable for some time. We also provide a Location.is_none which checks if a location is equal to Location.none for all versions of the compiler we currently support (i.e. >= 4.04). The decision to change Location.none has been made to enable the code to be simplified if we decide to change the lower bound of ppxlib to 4.08 in the future. Location.is_none has been used to fix loc_of_attribute which would sometimes miss attributes with none locations (like compiler inserted ocaml.doc attributes). Signed-off-by: Patrick Ferris <patrick@sirref.org>

Add Location.is_none and fix loc_of_attributes Location.is_none provides a means to check whether a given location is none for all supported compiler versions (as the none location changed). We then use this is loc_of_attributes to correctly pick-up none-located, compiler attributes like documentation comments. Signed-off-by: Patrick Ferris <patrick@sirref.org>

Add Location.is_none and fix loc_of_attributes Location.is_none provides a means to check whether a given location is none for all supported compiler versions (as the none location changed). We then use this is loc_of_attributes to correctly pick-up none-located, compiler attributes like documentation comments. Signed-off-by: Patrick Ferris <patrick@sirref.org>

Switch dummy_pos depending on compiler version In OCaml 4.08 the dummy position was changed to have a pos_lnum of 0 instead of 1. We need to match whatever the compiler is using. Signed-off-by: Patrick Ferris <patrick@sirref.org>

Switch dummy_pos depending on compiler version In OCaml 4.08 the dummy position was changed to have a pos_lnum of 0 instead of 1. We need to match whatever the compiler is using. Signed-off-by: Patrick Ferris <patrick@sirref.org>

Switch dummy_pos depending on compiler version In OCaml 4.08 the dummy position was changed to have a pos_lnum of 0 instead of 1. We need to match whatever the compiler is using. Signed-off-by: Patrick Ferris <patrick@sirref.org>

Add changes entry Signed-off-by: Patrick Ferris <patrick@sirref.org>

Update tests post dummy position change Signed-off-by: Patrick Ferris <patrick@sirref.org>

Update tests for dummy position update Signed-off-by: Patrick Ferris <patrick@sirref.org>