How to properly document changes made to someone else's code

Want to talk about something that isn't covered by another category?

Moderators: FourthWorld, heatherlaine, Klaus, kevinmiller, robinmiller

Post Reply
pink
Posts: 272
Joined: Wed Mar 12, 2014 6:18 pm

How to properly document changes made to someone else's code

Post by pink » Wed Jul 08, 2015 6:12 pm

I'm working on a library for CouchDB. I'm using the library EasyJSON for dealing with JSON documents.

I've made one very very small tweak in EasyJSON which overrides the function which converts numbered keys into a JSON array based on a custom property.

When I release this, what would be the proper way to document that I made a small change to someone else's library?

Code: Select all

/*
This function checks the keys of a LiveCode array,
and returns TRUE if all the keys are numerical - otherwise, returns false.

@param pArray a LiveCode array
@return TRUE if the array's keys are all numerical, otherwise FALSE
              UNLESS there is a customer property on the stack called numericOverride which has been set to TRUE 
*/
private function isNumericalArray pArray
     local tKeys,tOverride
     put the numericOverride of this stack into tOverride  ----added this
     if tOverride is false then return false   ----and this
     else
          put the keys of pArray into tKeys
          filter tKeys without "[0-9]*"
          if the number of lines in tKeys > 0 then return false
          else return true
     end if
end isNumericalArray
Greg (pink) Miller

MadPink, LLC
I'm Mad, Pink and Dangerous to Know

richmond62
Livecode Opensource Backer
Livecode Opensource Backer
Posts: 9388
Joined: Fri Feb 19, 2010 10:17 am
Location: Bulgaria

Re: How to properly document changes made to someone else's

Post by richmond62 » Wed Jul 08, 2015 6:27 pm

How about something like this:
addition.png

mwieder
VIP Livecode Opensource Backer
VIP Livecode Opensource Backer
Posts: 3581
Joined: Mon Jan 22, 2007 7:36 am
Location: Berkeley, CA, US
Contact:

Re: How to properly document changes made to someone else's

Post by mwieder » Wed Jul 08, 2015 10:21 pm

You might submit a pull request to the original code so that your improvements can be incorporated into the library...

https://github.com/luxlogica/easyjson

Post Reply

Return to “Off-Topic”