Opened 6 years ago

Closed 6 years ago

#3511 closed enhancement (fixed)

[patch] v.buffer: allow squared buffers around points

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 7.4.1
Component: Vector Version: svn-trunk
Keywords: v.buffer squared buffer Cc:
CPU: Unspecified Platform: Unspecified

Description

The attached patch implements squared buffers around points.

Attachments (2)

squared_buffer_around_points.diff (754 bytes ) - added by mlennert 6 years ago.
squared_buffer_around_points_directly_in_vbuffer.diff (1.3 KB ) - added by mlennert 6 years ago.
second version of a patch, this time directly in main.c of v.buffer

Download all attachments as: .zip

Change History (8)

by mlennert, 6 years ago

in reply to:  description ; comment:1 by mlennert, 6 years ago

Replying to mlennert:

The attached patch implements squared buffers around points.

Forgot to explain: with the patch, v.buffer should create square buffers around points if the -s flag is set.

in reply to:  1 ; comment:2 by mmetz, 6 years ago

Replying to mlennert:

Replying to mlennert:

The attached patch implements squared buffers around points.

Forgot to explain: with the patch, v.buffer should create square buffers around points if the -s flag is set.

Currently v.buffer always uses Vect_point_buffer2() for points, but the Vect_*_buffer*() functions might disappear at some stage because they are not working properly (apart from point buffering). Therefore it might be safer to add this functionality to v.buffer directly.

by mlennert, 6 years ago

second version of a patch, this time directly in main.c of v.buffer

in reply to:  2 ; comment:3 by mlennert, 6 years ago

Replying to mmetz:

Replying to mlennert:

Replying to mlennert:

The attached patch implements squared buffers around points.

Forgot to explain: with the patch, v.buffer should create square buffers around points if the -s flag is set.

Currently v.buffer always uses Vect_point_buffer2() for points, but the Vect_*_buffer*() functions might disappear at some stage because they are not working properly (apart from point buffering). Therefore it might be safer to add this functionality to v.buffer directly.

Please check the attached new patch.

in reply to:  3 ; comment:4 by mmetz, 6 years ago

Replying to mlennert:

Replying to mmetz:

Replying to mlennert:

Replying to mlennert:

The attached patch implements squared buffers around points.

Forgot to explain: with the patch, v.buffer should create square buffers around points if the -s flag is set.

Currently v.buffer always uses Vect_point_buffer2() for points, but the Vect_*_buffer*() functions might disappear at some stage because they are not working properly (apart from point buffering). Therefore it might be safer to add this functionality to v.buffer directly.

Please check the attached new patch.

The patch looks and works fine, please commit.

in reply to:  4 comment:5 by mlennert, 6 years ago

Replying to mmetz:

Replying to mlennert:

Replying to mmetz:

Replying to mlennert:

Replying to mlennert:

The attached patch implements squared buffers around points.

Forgot to explain: with the patch, v.buffer should create square buffers around points if the -s flag is set.

Currently v.buffer always uses Vect_point_buffer2() for points, but the Vect_*_buffer*() functions might disappear at some stage because they are not working properly (apart from point buffering). Therefore it might be safer to add this functionality to v.buffer directly.

Please check the attached new patch.

The patch looks and works fine, please commit.

Done in r72353.

comment:6 by mlennert, 6 years ago

Resolution: fixed
Status: newclosed

Closing this ticket as fixed.

Note: See TracTickets for help on using tickets.